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] product_set: Migration to 17.0 #1554

Merged
merged 5 commits into from
Mar 26, 2024

Conversation

NICO-SOLUTIONS
Copy link
Member

@NICO-SOLUTIONS NICO-SOLUTIONS commented Mar 15, 2024

migration to 17.0

superseeds #1493

@OCA-git-bot
Copy link
Contributor

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

Copy link
Contributor

@CRogos CRogos left a comment

Choose a reason for hiding this comment

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

Can you fix and cleanup the commit history?

@NICO-SOLUTIONS NICO-SOLUTIONS force-pushed the 17.0-mig-product_set_1 branch from 2543251 to f5ca78f Compare March 25, 2024 16:51
@NICO-SOLUTIONS
Copy link
Member Author

NICO-SOLUTIONS commented Mar 25, 2024

@CRogos
thanks for the hint. I did not realise that i have pushed 2 MIG-commits.

Christopher Rogos and others added 3 commits March 25, 2024 19:27
[UPD] Update product_set.pot

[UPD] README.rst

Translated using Weblate (Spanish)

Currently translated at 100.0% (23 of 23 strings)

Translation: product-attribute-16.0/product-attribute-16.0-product_set
Translate-URL: https://translation.odoo-community.org/projects/product-attribute-16-0/product-attribute-16-0-product_set/es/

[UPD] README.rst

Translated using Weblate (Portuguese (Brazil))

Currently translated at 100.0% (23 of 23 strings)

Translation: product-attribute-16.0/product-attribute-16.0-product_set
Translate-URL: https://translation.odoo-community.org/projects/product-attribute-16-0/product-attribute-16-0-product_set/pt_BR/

Update translation files

Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: product-attribute-16.0/product-attribute-16.0-product_set
Translate-URL: https://translation.odoo-community.org/projects/product-attribute-16-0/product-attribute-16-0-product_set/

Translated using Weblate (Italian)

Currently translated at 100.0% (23 of 23 strings)

Translation: product-attribute-16.0/product-attribute-16.0-product_set
Translate-URL: https://translation.odoo-community.org/projects/product-attribute-16-0/product-attribute-16-0-product_set/it/
[UPD] Update product_set.pot

[BOT] post-merge updates

Update translation files

Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: product-attribute-16.0/product-attribute-16.0-product_set
Translate-URL: https://translation.odoo-community.org/projects/product-attribute-16-0/product-attribute-16-0-product_set/

Translated using Weblate (Spanish)

Currently translated at 100.0% (29 of 29 strings)

Translation: product-attribute-16.0/product-attribute-16.0-product_set
Translate-URL: https://translation.odoo-community.org/projects/product-attribute-16-0/product-attribute-16-0-product_set/es/

Translated using Weblate (Italian)

Currently translated at 100.0% (29 of 29 strings)

Translation: product-attribute-16.0/product-attribute-16.0-product_set
Translate-URL: https://translation.odoo-community.org/projects/product-attribute-16-0/product-attribute-16-0-product_set/it/
… in the module sale_product_set.

The reason to move this logic is that there are other modules that extend
product_set as for example stock_product_set but at the same time make use
of the transient model to define a wizard. For this reason it is better to
have the logic available in this base module and avoid duplicating code or
inheriting from sale_product_set and what this implies in their respective
dependencies with the only need to extend the transient model.
In addition, the transient model is renamed to make it clearer to identify
that it is this type of model.

TT48100

[UPD] Update product_set.pot

[BOT] post-merge updates

Update translation files

Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: product-attribute-16.0/product-attribute-16.0-product_set
Translate-URL: https://translation.odoo-community.org/projects/product-attribute-16-0/product-attribute-16-0-product_set/

Translated using Weblate (Italian)

Currently translated at 100.0% (32 of 32 strings)

Translation: product-attribute-16.0/product-attribute-16.0-product_set
Translate-URL: https://translation.odoo-community.org/projects/product-attribute-16-0/product-attribute-16-0-product_set/it/
@NICO-SOLUTIONS NICO-SOLUTIONS force-pushed the 17.0-mig-product_set_1 branch from f5ca78f to 8a4f327 Compare March 25, 2024 18:28
@NICO-SOLUTIONS NICO-SOLUTIONS requested a review from CRogos March 25, 2024 18:32
Copy link
Contributor

@CRogos CRogos left a comment

Choose a reason for hiding this comment

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

Commit history and code looks good now.
Could you readd the readme info as markdown file, and I think there is an error in the xml view.

product_set/readme/USAGE.rst Outdated Show resolved Hide resolved
product_set/views/product_set.xml Outdated Show resolved Hide resolved
@NICO-SOLUTIONS NICO-SOLUTIONS force-pushed the 17.0-mig-product_set_1 branch from 8a4f327 to da6e1ab Compare March 25, 2024 22:11
@pedrobaeza pedrobaeza changed the title [MIG] product_set: Migration to 17.0 [17.0][MIG] product_set: Migration to 17.0 Mar 26, 2024
@pedrobaeza
Copy link
Member

/ocabot migration product_set

@OCA-git-bot OCA-git-bot added this to the 17.0 milestone Mar 26, 2024
@OCA-git-bot OCA-git-bot mentioned this pull request Mar 26, 2024
63 tasks
@NICO-SOLUTIONS NICO-SOLUTIONS force-pushed the 17.0-mig-product_set_1 branch 2 times, most recently from 5936133 to 66691f1 Compare March 26, 2024 18:04
@NICO-SOLUTIONS NICO-SOLUTIONS force-pushed the 17.0-mig-product_set_1 branch from 66691f1 to 8c6c6f8 Compare March 26, 2024 18:17
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.

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 17.0-ocabot-merge-pr-1554-by-pedrobaeza-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 002b7a7 into OCA:17.0 Mar 26, 2024
5 of 7 checks passed
@OCA-git-bot
Copy link
Contributor

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

@NICO-SOLUTIONS NICO-SOLUTIONS deleted the 17.0-mig-product_set_1 branch March 26, 2024 18:54
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.

5 participants