Skip to content

[MIG] survey_legal: migrate to 18.0#223

Open
layadelcarmen wants to merge 6 commits into
OCA:18.0from
layadelcarmen:18.0-migration-survey_legal
Open

[MIG] survey_legal: migrate to 18.0#223
layadelcarmen wants to merge 6 commits into
OCA:18.0from
layadelcarmen:18.0-migration-survey_legal

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.

Please also check the red condition of the CI.

Laya Rabasa added 5 commits May 26, 2026 07:02
- Remove hardcoded survey URL
- Replace deprecated 'text' action with 'fill'
- Remove unknown key 'isCheck' from final tour step
- Add explicit run: click/edit actions (required in Odoo 18 tour runner)
- Replace fill with edit for text inputs
- Add question wrapper prefix to matrix question triggers
- Fix typo 'your' -> 'you' in matrix question title
@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.

Just tested on runboat. Functional review: LGT

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants