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

[15.0] [MIG] website_sale_product_sort: MIgration to 15.0 #644

Merged
merged 18 commits into from
Dec 26, 2022

Conversation

CarlosRoca13
Copy link
Contributor

cc @Tecnativa TT36606

please @chienandalu @pedrobaeza review this

chienandalu and others added 18 commits May 23, 2022 12:12
Currently translated at 100.0% (11 of 11 strings)

Translation: e-commerce-12.0/e-commerce-12.0-website_sale_product_sort
Translate-URL: https://translation.odoo-community.org/projects/e-commerce-12-0/e-commerce-12-0-website_sale_product_sort/es/
Currently translated at 100.0% (11 of 11 strings)

Translation: e-commerce-12.0/e-commerce-12.0-website_sale_product_sort
Translate-URL: https://translation.odoo-community.org/projects/e-commerce-12-0/e-commerce-12-0-website_sale_product_sort/fr/
Currently translated at 90.9% (10 of 11 strings)

Translation: e-commerce-12.0/e-commerce-12.0-website_sale_product_sort
Translate-URL: https://translation.odoo-community.org/projects/e-commerce-12-0/e-commerce-12-0-website_sale_product_sort/nl/
Currently translated at 100.0% (11 of 11 strings)

Translation: e-commerce-12.0/e-commerce-12.0-website_sale_product_sort
Translate-URL: https://translation.odoo-community.org/projects/e-commerce-12-0/e-commerce-12-0-website_sale_product_sort/ca/
Currently translated at 100.0% (11 of 11 strings)

Translation: e-commerce-13.0/e-commerce-13.0-website_sale_product_sort
Translate-URL: https://translation.odoo-community.org/projects/e-commerce-13-0/e-commerce-13-0-website_sale_product_sort/ca/
Currently translated at 100.0% (11 of 11 strings)

Translation: e-commerce-13.0/e-commerce-13.0-website_sale_product_sort
Translate-URL: https://translation.odoo-community.org/projects/e-commerce-13-0/e-commerce-13-0-website_sale_product_sort/es/
Currently translated at 100.0% (11 of 11 strings)

Translation: e-commerce-13.0/e-commerce-13.0-website_sale_product_sort
Translate-URL: https://translation.odoo-community.org/projects/e-commerce-13-0/e-commerce-13-0-website_sale_product_sort/nl/
… is_published

POST data can contain an empty order, so previous code took that one as
the valid, which is not correct. We need to check if there's a value,
not only if the key exist.

We use also `is_published` as the field for sorting, the same as
upstream v13.
@pedrobaeza
Copy link
Member

/ocabot migration website_sale_product_sort

@OCA-git-bot OCA-git-bot added this to the 15.0 milestone May 23, 2022
@OCA-git-bot OCA-git-bot mentioned this pull request May 23, 2022
41 tasks
@github-actions
Copy link

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Dec 25, 2022
@OCA-git-bot
Copy link
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

@pedrobaeza
Copy link
Member

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

On my way to merge this fine PR!
Prepared branch 15.0-ocabot-merge-pr-644-by-pedrobaeza-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit d701633 into OCA:15.0 Dec 26, 2022
@OCA-git-bot
Copy link
Contributor

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

pilarvargas-tecnativa pushed a commit to Tecnativa/e-commerce that referenced this pull request Jan 24, 2023
Signed-off-by pedrobaeza
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved merged 🎉 ready to merge stale PR/Issue without recent activity, it'll be soon closed automatically.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants