[18.0][MIG] web_m2x_options_manager: Migration to 18.0 #2975
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
chaule97
commented
Oct 29, 2024
chaule97
commented
- Depends on: [18.0][MIG] web_m2x_options #2961
chaule97
force-pushed
the
18.0-mig-web_m2x_options_manager
branch
2 times, most recently
from
1475405 to
843bbd8
Compare
lucianobaleani
added a commit
to AITIC/web
that referenced
this pull request
Apr 30, 2025
|
@chaule97 the dependency is already merged and test-requirements gives and error because the pr is closed, Can you remove that file ? |
…ons from interface
- Fix postprocess inherit (removed from v15), and use _postprocess_tag_field for create/edit options: In v14 it worked with inheriting the "postprocess" method because in each recursive call it passed a section of the view, in the way that it went through all the child nodes of the root node, and passed through the fields, and now in v15 the method "_postprocess_view "it just does the while, and we never go through the nodes. - Fix pylint errors: xml-deprecated-tree-attributte, attribute-string-redundant, translation-positional-used
chaule97
force-pushed
the
18.0-mig-web_m2x_options_manager
branch
from
843bbd8 to
1cada9d
Compare
Author
I updated, could you review my PR? |
|
@chaule97 I'm going to do some tests to see if the module works well, and I'll give you the approve |
adrip-s73
approved these changes
Aug 7, 2025
Vicent-S73
pushed a commit
to Studio73/e-commerce
that referenced
this pull request
Aug 12, 2025
clementmbr
approved these changes
Sep 30, 2025
Member
clementmbr
left a comment
clementmbr
left a comment
There was a problem hiding this comment.
LGTM!
Is it possible to merge the PR?
|
@OCA/web-maintainers Please, can you merge this PR? |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
11 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.