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

[14.0][MIG] product_route_mto: backport from v16 #1724

Merged
merged 13 commits into from
Sep 24, 2024

Conversation

twalter-c2c
Copy link

@twalter-c2c twalter-c2c commented Sep 13, 2024

This module depends on stock_route_mto, here is the related PR: OCA/stock-logistics-warehouse#2154

@pedrobaeza
Copy link
Member

You should respect commit history.

@twalter-c2c
Copy link
Author

twalter-c2c commented Sep 13, 2024

Thank you for this remark, I will take care of it.

rousseldenis and others added 11 commits September 13, 2024 16:10
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: product-attribute-16.0/product-attribute-16.0-product_route_mto
Translate-URL: https://translation.odoo-community.org/projects/product-attribute-16-0/product-attribute-16-0-product_route_mto/
Currently translated at 100.0% (4 of 4 strings)

Translation: product-attribute-16.0/product-attribute-16.0-product_route_mto
Translate-URL: https://translation.odoo-community.org/projects/product-attribute-16-0/product-attribute-16-0-product_route_mto/it/
Currently translated at 100.0% (4 of 4 strings)

Translation: product-attribute-16.0/product-attribute-16.0-product_route_mto
Translate-URL: https://translation.odoo-community.org/projects/product-attribute-16-0/product-attribute-16-0-product_route_mto/it/
Currently translated at 100.0% (4 of 4 strings)

Translation: product-attribute-16.0/product-attribute-16.0-product_route_mto
Translate-URL: https://translation.odoo-community.org/projects/product-attribute-16-0/product-attribute-16-0-product_route_mto/es/
@sebalix
Copy link
Contributor

sebalix commented Sep 16, 2024

@twalter-c2c you can refer to the other PR temporarily to ensure your module is working as expected (tests + runboat) with this guide: https://github.com/OCA/maintainer-tools/wiki/Use-temporary-reference%28s%29-to-another-pull-request%28s%29

Once the dependency is merged, you could remove this reference from the current PR.

Copy link
Contributor

@ajaniszewska-dev ajaniszewska-dev left a comment

Choose a reason for hiding this comment

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

LG, test-requirements PR is already merged

@sebalix
Copy link
Contributor

sebalix commented Sep 24, 2024

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

Sorry @sebalix you are not allowed to merge.

To do so you must either have push permissions on the repository, or be a declared maintainer of all modified addons.

If you wish to adopt an addon and become it's maintainer, open a pull request to add your GitHub login to the maintainers key of its manifest.

@simahawk
Copy link
Contributor

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 14.0-ocabot-merge-pr-1724-by-simahawk-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 62567f0 into OCA:14.0 Sep 24, 2024
13 of 21 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at c5c3735. 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.