[16.0][ADD/MIG] l10n_ch_qr_bill_scan - #753
Conversation
a3b7368 to
57f3364
Compare
984a44b to
b78dce9
Compare
|
There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days. |
github-actions
Bot
added
the
stale
|
@OCA/local-switzerland-maintainers could you please re-open this PR? |
TDu
removed
the
stale
b78dce9 to
50e5f4a
Compare
18a3f85 to
d99dd8b
Compare
|
There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days. |
github-actions
Bot
added
the
stale
|
|
||
| state = fields.Selection( | ||
| selection_add=[("select-partner", "Select partner")], | ||
| selection=[("select-partner", "Select partner")], |
There was a problem hiding this comment.
The implementation of the account_invoice_import module has been improved by the author in OCA/edi#753
I think we should follow the same logic here and not add this state field that has been removed.
github-actions
Bot
removed
the
stale
c7843c2 to
7654cb2
Compare
A new module to scan QR-bill or to import PDF files
…ound but cant be readable
…hitecture - Remove state and partner-related fields from the wizard, as they are now handled by the base module via a banner/wizard on the created invoice. - Update import_invoices to handle the new multi-file attachment flow by creating an attachment from the invoice_scan field. - Refactor parse_pdf_invoice and parse_qrbill to match new signatures and simplify the parsing flow. - Improve plain text QR data detection to handle leading whitespace and different line endings. - Update tests to use new res.partner configuration fields and verify import_partner_data on invoices.
7654cb2 to
7aeba32
Compare
|
There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days. |
github-actions
Bot
added
the
stale
TDu
removed
the
stale
OCA-git-bot
added
series:16.0
mod:l10n_ch_qr_bill_scan
9 participants
This PR replaces #698
Due to the complicated history of this module, I'd like to clarify that this is both a migration and a module addition.
The module presented in #553 for 13.0 (then superseded by #718) was never merged but nonetheless migrated to different versions (14.0, 15.0, even backported to the 10.0
None of the PRs mentioned above have been merged. I hope I didn't miss any relevant context, but this is another attempt to bring this module to the main branch, making it a module addition—thus the branch name.