Skip to content

[19.0][OU-ADD] survey: Migration scripts#5760

Merged
pedrobaeza merged 1 commit into
OCA:19.0from
Tecnativa:19.0-ou-add-survey
Jun 30, 2026
Merged

[19.0][OU-ADD] survey: Migration scripts#5760
pedrobaeza merged 1 commit into
OCA:19.0from
Tecnativa:19.0-ou-add-survey

Conversation

@carlos-lopez-tecnativa

Copy link
Copy Markdown
Contributor

TT63304
@Tecnativa @pedrobaeza could you please review this?

@OCA-git-bot OCA-git-bot added mod:openupgrade_scripts Module openupgrade_scripts series:19.0 labels Jun 26, 2026
@legalsylvain

legalsylvain commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

/ocabot migration survey

Depends on :

@OCA-git-bot OCA-git-bot added this to the 19.0 milestone Jun 26, 2026

@pedrobaeza pedrobaeza left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@eduezerouali-tecnativa please perform the changes, as Carlos is away these days.

---Fields in module 'survey'---
survey / survey.survey / lang_ids (many2many) : NEW relation: res.lang, hasdefault: default
survey / survey.user_input / lang_id (many2one) : NEW relation: res.lang
# NOTHING TO DO: Leave lang_ids empty to use all installed languages.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This is not what the default does, as it assigns in lang_ids the first installed language found:

https://github.com/odoo/odoo/blob/161715c850496d3683baa5d1600380470d0b5ff5/addons/survey/models/survey_survey.py#L49

In post-migration, just delete whole m2m table for getting the expected result.

Comment thread openupgrade_scripts/scripts/survey/19.0.3.7/post-migration.py Outdated
@pedrobaeza pedrobaeza merged commit dcc1376 into OCA:19.0 Jun 30, 2026
7 checks passed
@pedrobaeza pedrobaeza deleted the 19.0-ou-add-survey branch June 30, 2026 11:35
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.

6 participants