Skip to content

Conversation

@Abranes
Copy link
Member

@Abranes Abranes commented Nov 9, 2025

Migration to 18.0 from #1331

legalsylvain and others added 30 commits November 9, 2025 21:46
- Black python code
- OCA Convention
- Add tests
- add configuration on pos.config, with two option 'refund' or 'update'
- The 'refund' option makes the module compatible with (French) certification
- make the module compatible with pos_order_return
…on PoS Order(s) to know the history of the payments
Currently translated at 100.0% (31 of 31 strings)

Translation: pos-12.0/pos-12.0-pos_payment_change
Translate-URL: https://translation.odoo-community.org/projects/pos-12-0/pos-12-0-pos_payment_change/es/
Currently translated at 100.0% (33 of 33 strings)

Translation: pos-12.0/pos-12.0-pos_payment_change
Translate-URL: https://translation.odoo-community.org/projects/pos-12-0/pos-12-0-pos_payment_change/es/
Currently translated at 100.0% (33 of 33 strings)

Translation: pos-12.0/pos-12.0-pos_payment_change
Translate-URL: https://translation.odoo-community.org/projects/pos-12-0/pos-12-0-pos_payment_change/de/
A cash payment method must have a cash journal.
Currently translated at 100.0% (33 of 33 strings)

Translation: pos-16.0/pos-16.0-pos_payment_change
Translate-URL: https://translation.odoo-community.org/projects/pos-16-0/pos-16-0-pos_payment_change/es/
Currently translated at 100.0% (33 of 33 strings)

Translation: pos-16.0/pos-16.0-pos_payment_change
Translate-URL: https://translation.odoo-community.org/projects/pos-16-0/pos-16-0-pos_payment_change/it/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: pos-16.0/pos-16.0-pos_payment_change
Translate-URL: https://translation.odoo-community.org/projects/pos-16-0/pos-16-0-pos_payment_change/
Currently translated at 100.0% (33 of 33 strings)

Translation: pos-16.0/pos-16.0-pos_payment_change
Translate-URL: https://translation.odoo-community.org/projects/pos-16-0/pos-16-0-pos_payment_change/es/
Currently translated at 100.0% (33 of 33 strings)

Translation: pos-16.0/pos-16.0-pos_payment_change
Translate-URL: https://translation.odoo-community.org/projects/pos-16-0/pos-16-0-pos_payment_change/fr/
In old version, two certifications modules was present in odoo.  l10n_fr_pos_cert and l10n_fr_certification.
In odoo 16, only l10n_fr_pos_cert exists.
@Abranes Abranes force-pushed the 18.0-mig-pos_payment_change branch from 0fb2a05 to 4c9439a Compare November 9, 2025 22:25
Copy link

@legalsylvain legalsylvain left a comment

Choose a reason for hiding this comment

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

Diff review. LGTM. Thanks !

@pedrobaeza
Copy link
Member

/ocabot migration pos_payment_change

@OCA-git-bot OCA-git-bot added this to the 18.0 milestone Nov 10, 2025
@OCA-git-bot OCA-git-bot mentioned this pull request Nov 10, 2025
16 tasks
@pedrobaeza
Copy link
Member

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 18.0-ocabot-merge-pr-1453-by-pedrobaeza-bump-nobump, awaiting test results.

OCA-git-bot added a commit that referenced this pull request Nov 10, 2025
Signed-off-by pedrobaeza
@OCA-git-bot
Copy link
Contributor

@pedrobaeza your merge command was aborted due to failed check(s), which you can inspect on this commit of 18.0-ocabot-merge-pr-1453-by-pedrobaeza-bump-nobump.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

@pedrobaeza
Copy link
Member

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

On my way to merge this fine PR!
Prepared branch 18.0-ocabot-merge-pr-1453-by-pedrobaeza-bump-nobump, awaiting test results.

OCA-git-bot added a commit that referenced this pull request Nov 10, 2025
Signed-off-by pedrobaeza
@OCA-git-bot
Copy link
Contributor

@pedrobaeza your merge command was aborted due to failed check(s), which you can inspect on this commit of 18.0-ocabot-merge-pr-1453-by-pedrobaeza-bump-nobump.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

Copy link
Member

@pedrobaeza pedrobaeza left a comment

Choose a reason for hiding this comment

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

Only OCB is failing due to a tour... let's try again

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 18.0-ocabot-merge-pr-1453-by-pedrobaeza-bump-nobump, awaiting test results.

OCA-git-bot added a commit that referenced this pull request Nov 10, 2025
Signed-off-by pedrobaeza
@OCA-git-bot
Copy link
Contributor

@pedrobaeza your merge command was aborted due to failed check(s), which you can inspect on this commit of 18.0-ocabot-merge-pr-1453-by-pedrobaeza-bump-nobump.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

@pedrobaeza
Copy link
Member

There's no way...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.