Skip to content

Conversation

d-fence
Copy link
Contributor

@d-fence d-fence commented Mar 31, 2025

The previous fix in #204089 didn't work as expected because the tour_enabled field is depending on the creation date. As the field is stored and the base user is created before the start of the tests, the demo tours are enabled for the click_all for admin.

With this commit, the field is explicitly set to False before starting the click all for admin.

The previous fix in odoo#204089 didn't work as expected because the
`tour_enabled` field is depending on the creation date. As the field is
stored and the base user is created before the start of the tests, the
demo tours are enabled for the click_all for admin.

With this commit, the field is explicitly set to False before starting
the click all for admin.
@robodoo
Copy link
Contributor

robodoo commented Mar 31, 2025

Pull request status dashboard

@d-fence d-fence requested a review from Xavier-Do March 31, 2025 12:17
@C3POdoo C3POdoo added the RD research & development, internal work label Mar 31, 2025
@d-fence d-fence marked this pull request as ready for review March 31, 2025 14:11
@C3POdoo C3POdoo requested review from a team, aab-odoo and kebeclibre and removed request for a team March 31, 2025 14:14
@d-fence
Copy link
Contributor Author

d-fence commented Mar 31, 2025

robodoo r+

robodoo pushed a commit that referenced this pull request Mar 31, 2025
The previous fix in #204089 didn't work as expected because the
`tour_enabled` field is depending on the creation date. As the field is
stored and the base user is created before the start of the tests, the
demo tours are enabled for the click_all for admin.

With this commit, the field is explicitly set to False before starting
the click all for admin.

closes #204145

Signed-off-by: Christophe Monniez (moc) <moc@odoo.com>
@robodoo robodoo added the 18.3 label Mar 31, 2025
@robodoo robodoo closed this Mar 31, 2025
@d-fence d-fence deleted the master-better-no-purple-roupettes-moc branch April 1, 2025 08:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

18.3 RD research & development, internal work

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants