Skip to content

[18.0][IMP] base_import_pdf_by_template: Add date_time_format field to improve all formats#1349

Open
victoralmau wants to merge 1 commit into
OCA:18.0from
Tecnativa:18.0-imp-base_import_pdf_by_template-misc
Open

[18.0][IMP] base_import_pdf_by_template: Add date_time_format field to improve all formats#1349
victoralmau wants to merge 1 commit into
OCA:18.0from
Tecnativa:18.0-imp-base_import_pdf_by_template-misc

Conversation

@victoralmau

Copy link
Copy Markdown
Member

Add date_time_format field to improve all formats

Please @pedrobaeza and @cristina-hidalgo-tecnativa can you review it?

@Tecnativa

@OCA-git-bot OCA-git-bot added series:18.0 mod:base_import_pdf_by_template Module base_import_pdf_by_template labels May 25, 2026
@pedrobaeza pedrobaeza added this to the 18.0 milestone May 26, 2026
Comment thread base_import_pdf_by_template/migrations/18.0.1.1.0/pre-migration.py Outdated
@victoralmau
victoralmau force-pushed the 18.0-imp-base_import_pdf_by_template-misc branch from 22a9511 to c5c5716 Compare June 1, 2026 15:24

@pedrobaeza pedrobaeza left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The problem with this change is that you need to know the syntax, so maybe a help explaining each possible parameter can be good.

@victoralmau
victoralmau force-pushed the 18.0-imp-base_import_pdf_by_template-misc branch from c5c5716 to 5d35765 Compare June 17, 2026 16:33
@victoralmau
victoralmau requested a review from pedrobaeza June 18, 2026 06:01
@pedrobaeza

Copy link
Copy Markdown
Member

I don't see the help of the field talking about the possible symbols to put (Y, d, m...).

@victoralmau
victoralmau force-pushed the 18.0-imp-base_import_pdf_by_template-misc branch from 5d35765 to 79d4d8a Compare June 24, 2026 14:59
Comment thread base_import_pdf_by_template/models/base_import_pdf_template.py Outdated
@victoralmau
victoralmau force-pushed the 18.0-imp-base_import_pdf_by_template-misc branch from 79d4d8a to d93fda1 Compare July 20, 2026 09:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

mod:base_import_pdf_by_template Module base_import_pdf_by_template series:18.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants