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

[16.0] [ADD] shopinvader_product_sale_packaging #1415

Conversation

marielejeune
Copy link
Contributor

@marielejeune marielejeune commented Oct 3, 2023

Add packaging info on products exports. Extracted from shopinvader_sale_packaging

NB: When migrating from v14 to v16 model product.packaging.type was renamed product.packaging.level

@marielejeune marielejeune marked this pull request as draft October 3, 2023 08:34
@marielejeune marielejeune force-pushed the 16.0-shopinvader_product_sale_packaging branch 3 times, most recently from fffb712 to b583dd6 Compare October 3, 2023 11:33
@marielejeune
Copy link
Contributor Author

@sbidoul @lmignon tests of shopinvader_anonymous_partner are failing, do you have an idea why?
This addon shouldn't be concerned by this PR...

@marielejeune marielejeune force-pushed the 16.0-shopinvader_product_sale_packaging branch from b583dd6 to 3d4cf49 Compare October 3, 2023 13:27
@marielejeune marielejeune force-pushed the 16.0-shopinvader_product_sale_packaging branch 2 times, most recently from b8abb5b to d5ad128 Compare October 6, 2023 10:23
@marielejeune marielejeune marked this pull request as ready for review October 6, 2023 10:23
@marielejeune marielejeune force-pushed the 16.0-shopinvader_product_sale_packaging branch 4 times, most recently from 042e8a4 to caad935 Compare October 9, 2023 11:04
@marielejeune
Copy link
Contributor Author

@sbidoul test_create_and_get in shopinvader_anonymous_partner is failing because of this Odoo bug: odoo/odoo#137997

Do you see a way to make this pipeline succeed? The company is lost from the context when calling url_open.

@sbidoul
Copy link
Member

sbidoul commented Oct 9, 2023

🍏

@simahawk
Copy link
Contributor

@marielejeune all deps got merged ;)

marielejeune and others added 2 commits December 19, 2023 08:21
With auth=none, request.env does not have a user,
and therefore no company_id.
Use the super user to avoid issues in
other addons that expect env.company to be
defined.
@marielejeune marielejeune force-pushed the 16.0-shopinvader_product_sale_packaging branch from 345e095 to 221a215 Compare December 19, 2023 07:23
@simahawk
Copy link
Contributor

can we merge or extrapolate the fix by @sbidoul ?

@lmignon
Copy link
Collaborator

lmignon commented Dec 19, 2023

/ocabot merge patch

@shopinvader-git-bot
Copy link

What a great day to merge this nice PR. Let's do it!
Prepared branch 16.0-ocabot-merge-pr-1415-by-lmignon-bump-patch, awaiting test results.

@shopinvader-git-bot shopinvader-git-bot merged commit 1e56c7b into shopinvader:16.0 Dec 19, 2023
3 checks passed
@shopinvader-git-bot
Copy link

Congratulations, your PR was merged at 2e20821. Thanks a lot for contributing to shopinvader. ❤️

@sbidoul sbidoul deleted the 16.0-shopinvader_product_sale_packaging branch December 19, 2023 16:51
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