Skip to content

[MIG] survey_skip_start: migrate to 18.0#222

Open
layadelcarmen wants to merge 3 commits into
OCA:18.0from
layadelcarmen:18.0-migration-survey_skip_start
Open

[MIG] survey_skip_start: migrate to 18.0#222
layadelcarmen wants to merge 3 commits into
OCA:18.0from
layadelcarmen:18.0-migration-survey_skip_start

Conversation

@layadelcarmen
Copy link
Copy Markdown

No description provided.

@pedrobaeza
Copy link
Copy Markdown
Member

Thanks for the contribution.

Please preserve commit history following technical method explained in https://github.com/OCA/maintainer-tools/wiki/Migration-to-version-18.0.

If the jump is between several versions, you have to modify the source branch in the main command to accommodate it to this circumstance.

Laya Rabasa added 2 commits May 26, 2026 05:02
…ibility

In Odoo 18 the tour runner uses the tour's url field as startUrl, overriding
the dynamic token passed by start_tour(). Removing it lets the runner stay
on the URL already navigated to by the test.
The text action helper was renamed to fill in Odoo 18's tour runner.
@layadelcarmen
Copy link
Copy Markdown
Author

I noticed the CI run is failing with a warning related to Chrome (a "Killing chrome descendants-or-self" message picked up by oca_checklog_odoo). I haven't been able to reproduce this locally and it doesn't seem related to the module code itself, but I wanted to flag it in case it's a known issue or if there's something specific I should do on my end to address it. Thank you in advance.

@layadelcarmen
Copy link
Copy Markdown
Author

@benbrich could you review this PR please?

Copy link
Copy Markdown
Contributor

@benbrich benbrich left a comment

Choose a reason for hiding this comment

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

Tested on runboat. Functionally LGT

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

mod:survey_skip_start Module survey_skip_start series:18.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants