-
-
Notifications
You must be signed in to change notification settings - Fork 311
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
[15.0] FWD port of multiple PRs #814
Conversation
Hi @simahawk, |
3a72b1d
to
0f2904b
Compare
@etobella we must add a blacklist file for oca-port. I'll add one for v16. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
TL;DR But it's green 🟢
We need to fix the tests on 15.0 branch.
So 🤞 on this one
@simahawk shall we proceed with merge? |
Wow, "small list of PRs" having 85 files changed. I wouldn't call it small... Should I proceed with the merge? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@etobella IMO there are too many things here... included the migration of some modules, like 0f2904b
We cannot even bump properly the versions...
I don't think we need everything to fix the build.
I've just fixed the issue on v16 here OCA/edi-framework@f1e6976 and I will rework the fix for v14 and v15.
I'll check in a while
conflicts |
You can now track the original exchange record that originated the document in the 1st place.
Ease finding which type the current one is an ack for.
You can produce or receive more than one ack for a given exchange. If that's the case, make sure only the newest is considered.
In case exchange_on is the same, use ID to get the latest on top.
Skip 'output_sent' records by default. Reason: most of the times when a message is sent out you don't care about its state anymore. Any subsequent update from outside should come with a new input record instead. Yet, if for any reason this feature is required you can still enable it by passing 'skip_sent=False'.
In this commit OCA@0a9b6d5 I unified the way these kind of methods were used. Was breaking older implementations where the method was called directly.
Long awaited improvement. Allows to decouple rules by model giving much more flexibility. Migration steps included.
You can now customize and translate labels and tooltips for EDI automatic buttons.
Avoid warning because of duplicated Model field name while installing this module
Use `name()` because the real element could be namespaced on render leading to mismatch by element type.
Custom kw args for rendering should be injected before snippets are evaluated. This way they are available for the snippet but also the snippet can override any custom param.
Use the full name of the partner by default to include parent name. This behavior can be tweaked w/ the work context key 'party_data_name_field'.
Small list of PRs pending to port to 15.0 😉
#639
#651
#652
#653
#645
#648
#662
#666
#672
#694
#696
#697
#698
#721
#762
#764
#763
#765
#767
#774
#778
#797
#790
#808