Skip to content

[20.0][MIG] web_quick_start_screen: Migration to 20.0 - #3762

Merged
OCA-git-bot merged 8 commits into
OCA:20.0from
Tecnativa:20.0-mig-web_quick_start_screen
Oct 29, 2026
Merged

OCA-git-bot merged 8 commits into
OCA:20.0from
Tecnativa:20.0-mig-web_quick_start_screen

Conversation

@CarlosRoca13

Copy link
Copy Markdown
Contributor

@OCA-git-bot OCA-git-bot added mod:web_quick_start_screen Module web_quick_start_screen series:20.0 labels Oct 28, 2026
@pedrobaeza

Copy link
Copy Markdown
Member

/ocabot migration web_quick_start_screen

@OCA-git-bot OCA-git-bot added this to the 20.0 milestone Oct 28, 2026
@OCA-git-bot OCA-git-bot mentioned this pull request Oct 28, 2026
4 of 36 tasks

@Andrii9090-tecnativa Andrii9090-tecnativa left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I created a new screen actions and added it to the Demo start screen.
Image

The action appears correctly in the screen's Actions field, but it is not displayed on the start screen.

Image

After installing another app, the previously missing actions appear on the start screen, I think this is related to the ORM cache.

Image

Could you review please?

@carlos-lopez-tecnativa carlos-lopez-tecnativa left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm a bit confused about the behavior. Let me explain:

With the admin user, whether the Quick Start Screen field is set or empty, the start action is always shown when I log in:

2026-10-28.10-45-30.mp4

But with the demo user, I set the Quick Start Screen field, but this action is never shown when I log in. Why?

2026-10-28.10-47-17.mp4

Comment thread web_quick_start_screen/migrations/20.0.1.0.0/post-migration.py Outdated
@CarlosRoca13
CarlosRoca13 force-pushed the 20.0-mig-web_quick_start_screen branch from 0c12e4f to 1c04a7b Compare October 29, 2026 06:42
@oca-cla-bot

oca-cla-bot Bot commented Oct 29, 2026

Copy link
Copy Markdown

The following commit authors need to sign the Contributor License Agreement:

CLA not signed

@CarlosRoca13

Copy link
Copy Markdown
Contributor Author

Thanks both for the review!

@Andrii9090-tecnativa You were right, it was the ORM cache. _visible_action_ids() is cached with @ormcache, but nothing cleared it when a screen action was created, archived or deleted. It only got refreshed when some core model cleared the default cache as a side effect, e.g. creating ir.actions.* records while installing a module. That's why the actions showed up after installing another app.

@carlos-lopez-tecnativa, logging in to the start screen depends on the user's Home Action (action_id), not on Quick Start Screen, which only picks the screen to show. In 20.0 core removed action_id from the user form, so it couldn't be set from the UI. Admin only had it through the demo data, which is why it always landed there (even with an empty screen). The demo user had no way to get it. I've added Home Action to the "Menus Customization" group of the user form (debug mode), next to Quick Start Screen, and updated CONFIGURE.md.

@Andrii9090-tecnativa Andrii9090-tecnativa left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! It works now, newly added actions appear on the start screen immediately.

@pedrobaeza

Copy link
Copy Markdown
Member

/ocabot merge nobump

@OCA-git-bot

Copy link
Copy Markdown
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 20.0-ocabot-merge-pr-3762-by-pedrobaeza-bump-nobump, awaiting test results.

@OCA-git-bot
OCA-git-bot merged commit 3a77e69 into OCA:20.0 Oct 29, 2026
5 of 8 checks passed
@OCA-git-bot

Copy link
Copy Markdown
Contributor

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

@carlos-lopez-tecnativa
carlos-lopez-tecnativa deleted the 20.0-mig-web_quick_start_screen branch October 29, 2026 13:49
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.

10 participants