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

[17.0][MIG] web_m2x_options_manager: Migration to 17.0 #2842

Open
wants to merge 15 commits into
base: 17.0
Choose a base branch
from

Conversation

Wodran14
Copy link

@Wodran14 Wodran14 commented Jun 5, 2024

No description provided.

SilvioC2C and others added 14 commits June 5, 2024 10:34
- 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
Currently translated at 100.0% (33 of 33 strings)

Translation: web-15.0/web-15.0-web_m2x_options_manager
Translate-URL: https://translation.odoo-community.org/projects/web-15-0/web-15-0-web_m2x_options_manager/es/
@Wodran14 Wodran14 marked this pull request as draft June 5, 2024 12:01
@Wodran14 Wodran14 marked this pull request as ready for review June 11, 2024 11:28
Copy link

@aliciagaarzo aliciagaarzo left a comment

Choose a reason for hiding this comment

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

The problem found in migration to v16 is not being solved

#2587 (review)

Is it worth adding a solution to this problem? Greetings.

@QuocDuong1306
Copy link

Hello @Wodran14 , The migration to v16 of this module is updated

Could you have a look and test it? If everything is ok, could you include them in your PR?

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

Successfully merging this pull request may close these issues.

8 participants