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][IMP] product_main_supplierinfo: replace _get_sellers by std _get_filtered_sellers method #1812

Merged
merged 1 commit into from
Dec 12, 2024

Conversation

sebalix
Copy link
Contributor

@sebalix sebalix commented Dec 10, 2024

Starting from v15, Odoo proposes the _get_filtered_sellers method that could be re-used to get sellers.

@sebalix sebalix force-pushed the 17-product_main_supplierinfo-imp branch from e55e12a to c0e47f7 Compare December 10, 2024 16:37
@sebalix sebalix changed the title product_main_supplierinfo: replace _get_sellers by std _get_filtered_sellers method [17.0][IMP] product_main_supplierinfo: replace _get_sellers by std _get_filtered_sellers method Dec 10, 2024
@xaviedoanhduy
Copy link

xaviedoanhduy commented Dec 11, 2024

hi @sebalix, I have create this PR to improve your changes.

@sebalix sebalix force-pushed the 17-product_main_supplierinfo-imp branch from c0e47f7 to d797363 Compare December 11, 2024 10:01
@sebalix
Copy link
Contributor Author

sebalix commented Dec 11, 2024

Hi @xaviedoanhduy , thanks for the followup! I took your commit to replace mine here (to preserve co-authorship).

@rousseldenis
Copy link
Contributor

Seems promising!

@xaviedoanhduy xaviedoanhduy force-pushed the 17-product_main_supplierinfo-imp branch from d797363 to b76052f Compare December 11, 2024 11:23
@xaviedoanhduy
Copy link

hi @sebalix, changed according to your suggestion.
thanks

@sebalix sebalix marked this pull request as ready for review December 12, 2024 09:56
@sebalix
Copy link
Contributor Author

sebalix commented Dec 12, 2024

@xaviedoanhduy thank you for the update, it looks good. I cannot review my own PR but I approve your changes.

@sebalix
Copy link
Contributor Author

sebalix commented Dec 12, 2024

/ocabot merge patch

@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-1812-by-sebalix-bump-patch, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 21b0c68 into OCA:17.0 Dec 12, 2024
7 checks passed
@OCA-git-bot
Copy link
Contributor

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

@sebalix sebalix deleted the 17-product_main_supplierinfo-imp branch December 13, 2024 12:40
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.

4 participants