Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[12.0][BKP] edi_oca: Backport from 14.0 #877

Merged
merged 365 commits into from
Mar 15, 2024

Conversation

MiquelRForgeFlow
Copy link
Contributor

Supersedes #802 and #860.

@OCA-git-bot
Copy link
Contributor

Hi @simahawk,
some modules you are maintaining are being modified, check this out!

@MiquelRForgeFlow MiquelRForgeFlow force-pushed the 12.0-true-backport branch 3 times, most recently from ed4aca7 to 0dc7784 Compare November 29, 2023 06:07
etobella and others added 26 commits November 30, 2023 12:07
Currently translated at 56.5% (78 of 138 strings)

Translation: edi-14.0/edi-14.0-edi_oca
Translate-URL: https://translation.odoo-community.org/projects/edi-14-0/edi-14-0-edi_oca/fr/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: edi-14.0/edi-14.0-edi_oca
Translate-URL: https://translation.odoo-community.org/projects/edi-14-0/edi-14-0-edi_oca/
use `_notify_*` methods when processing a exchange record. This
will allow the use of listeners based on this events.
This action allows to re-schedule the record for the action that failed
depending on its current state and direction.
@MiquelRForgeFlow MiquelRForgeFlow force-pushed the 12.0-true-backport branch 3 times, most recently from 07a0ced to 80dffe4 Compare November 30, 2023 12:27
@QuocDuong1306
Copy link

Hello @MiquelRForgeFlow , thanks for your effort!, could you have a look at tests (it's still red now)?

@MiquelRForgeFlow
Copy link
Contributor Author

@QuocDuong1306 if you can help me solve those errors, it would be nice :)

@QuocDuong1306
Copy link

QuocDuong1306 commented Jan 17, 2024

Thank @MiquelRForgeFlow, Sure, I'm going to have a look and prepare a PR for your branch

@QuocDuong1306
Copy link

Hello @MiquelRForgeFlow ,

I have done the fix 1 on this: OCA/queue#617 (merged)

Could you pick my commit about account_invoice_facturx: b4e0c6b
image

Hard override of _post_pdf in account_facturx makes the test fail.
@MiquelRForgeFlow
Copy link
Contributor Author

@simahawk could you merge?

@simahawk
Copy link
Contributor

simahawk commented Mar 15, 2024

Can we merge w/ patch or you are relying on any upgrade step for specific versions?
Otherwise you have to bump other modules versions manually (the ones not migrated I mean).
Sorry, I have no time to review everything here 😅

@MiquelRForgeFlow
Copy link
Contributor Author

MiquelRForgeFlow commented Mar 15, 2024

Yeah, you can merge w/ patch.

@simahawk
Copy link
Contributor

/ocabot merge patch

@OCA-git-bot
Copy link
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 12.0-ocabot-merge-pr-877-by-simahawk-bump-patch, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 6b5928f into OCA:12.0 Mar 15, 2024
7 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 9144e59. Thanks a lot for contributing to OCA. ❤️

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

Successfully merging this pull request may close these issues.