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

[FIX] web: fallback orderedBy from datapoint #31368

Open
wants to merge 1 commit into
base: 11.0
from

Conversation

Projects
None yet
4 participants
@kebeclibre
Copy link
Contributor

kebeclibre commented Feb 22, 2019

Have a list/kanban with a default_order on the view
Have a default filter that applies

Click on a element of the list
Go back to the list with the breadcrumbs

Before this commit, the default_order was not applied
This is because the wrong check was made to determine the orderedBy of the model

After this commit, the previous order that has been set on the model is kept
when reloading the model

OPW 1922576

Description of the issue/feature this PR addresses:

Current behavior before PR:

Desired behavior after PR is merged:

--
I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr

[FIX] web: fallback orderedBy from datapoint
Have a list/kanban with a default_order on the view
Have a default filter that applies

Click on a element of the list
Go back to the list with the breadcrumbs

Before this commit, the default_order was not applied
This is because the wrong check was made to determine the orderedBy of the model

After this commit, the previous order that has been set on the model is kept
when reloading the model

OPW 1922576

@kebeclibre kebeclibre requested a review from aab-odoo Feb 22, 2019

@robodoo robodoo added the seen 🙂 label Feb 22, 2019

@C3POdoo C3POdoo added the OE label Feb 22, 2019

@robodoo robodoo added the CI 🤖 label Feb 22, 2019

@aab-odoo
Copy link
Contributor

aab-odoo left a comment

I can't reproduce the issue. We should talk about this one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.