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

[FW][FIX] web: restore offset in grouped list #163068

Conversation

fw-bot
Copy link
Contributor

@fw-bot fw-bot commented Apr 23, 2024

Issue:
Have a grouped list view with several page, go to the next page, open a group and click on a record to open it in form view. Click on the breadcrumb to go back to the list: the offset is lost, and we're back in page 1.

After this commit, the offset is correctly kept.

opw~3851390

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

Forward-Port-Of: #162969
Forward-Port-Of: #162844

Issue:
Have a grouped list view with several pages, go to the next page,
open a group and click on a record to open it in form view. Click
on the breadcrumb to go back to the list: the offset is lost, and
we're back in page 1.

After this commit, the offset is correctly kept.

opw~3851390

X-original-commit: 8981338
@robodoo
Copy link
Contributor

robodoo commented Apr 23, 2024

@fw-bot
Copy link
Contributor Author

fw-bot commented Apr 23, 2024

@aab-odoo @SimonGenin this PR targets master and is the last of the forward-port chain containing:

To merge the full chain, use

@fw-bot r+

More info at https://github.com/odoo/odoo/wiki/Mergebot#forward-port

@robodoo robodoo added the forwardport This PR was created by @fw-bot label Apr 23, 2024
@C3POdoo C3POdoo added the OE the report is linked to a support ticket (opw-...) label Apr 23, 2024
@aab-odoo
Copy link
Contributor

@fw-bot r+

robodoo pushed a commit that referenced this pull request Apr 24, 2024
Issue:
Have a grouped list view with several pages, go to the next page,
open a group and click on a record to open it in form view. Click
on the breadcrumb to go back to the list: the offset is lost, and
we're back in page 1.

After this commit, the offset is correctly kept.

opw~3851390

closes #163068

X-original-commit: 8981338
Signed-off-by: Simon Genin (ges@odoo) <ges@odoo.com>
Signed-off-by: Aaron Bohy (aab) <aab@odoo.com>
@robodoo
Copy link
Contributor

robodoo commented Apr 24, 2024

@aab-odoo @SimonGenin staging failed: ci/runbot (view more at https://runbot.odoo.com/runbot/build/61747636)

@aab-odoo
Copy link
Contributor

@xmo-odoo Could you check this PR? I don't know what fw-bot is doing...

@xmo-odoo
Copy link
Collaborator

@aab-odoo I don't see anything to do with the fwbot? It was staged, and the staging failed. If the failure seems legit you fix the pr and re-approve, if it's not you retry the PR. That's normal PR flow.

@aab-odoo
Copy link
Contributor

robodoo retry

@aab-odoo
Copy link
Contributor

@xmo-odoo Ok my bad, I get confused by the linked (failing) build

@xmo-odoo
Copy link
Collaborator

Yeah it looks weird, might be because of the recent bootstrap updates in runbot /cc @d-fence @Xavier-Do

robodoo pushed a commit that referenced this pull request Apr 26, 2024
Issue:
Have a grouped list view with several pages, go to the next page,
open a group and click on a record to open it in form view. Click
on the breadcrumb to go back to the list: the offset is lost, and
we're back in page 1.

After this commit, the offset is correctly kept.

opw~3851390

closes #163068

X-original-commit: 8981338
Signed-off-by: Simon Genin (ges@odoo) <ges@odoo.com>
Signed-off-by: Aaron Bohy (aab) <aab@odoo.com>
robodoo pushed a commit that referenced this pull request Apr 26, 2024
Issue:
Have a grouped list view with several pages, go to the next page,
open a group and click on a record to open it in form view. Click
on the breadcrumb to go back to the list: the offset is lost, and
we're back in page 1.

After this commit, the offset is correctly kept.

opw~3851390

closes #163068

X-original-commit: 8981338
Signed-off-by: Simon Genin (ges@odoo) <ges@odoo.com>
Signed-off-by: Aaron Bohy (aab) <aab@odoo.com>
robodoo pushed a commit that referenced this pull request Apr 26, 2024
Issue:
Have a grouped list view with several pages, go to the next page,
open a group and click on a record to open it in form view. Click
on the breadcrumb to go back to the list: the offset is lost, and
we're back in page 1.

After this commit, the offset is correctly kept.

opw~3851390

closes #163068

X-original-commit: 8981338
Signed-off-by: Simon Genin (ges@odoo) <ges@odoo.com>
Signed-off-by: Aaron Bohy (aab) <aab@odoo.com>
@robodoo robodoo closed this Apr 26, 2024
@robodoo robodoo added the 17.3 label Apr 26, 2024
@Xavier-Do
Copy link
Contributor

This one https://runbot.odoo.com/runbot/build/61747636 ? What's the issue?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
17.3 forwardport This PR was created by @fw-bot OE the report is linked to a support ticket (opw-...)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants