Skip to content

[19.0][MIG] survey_question_type_binary#216

Open
glitchov wants to merge 13 commits into
OCA:19.0from
acsone:19.0-mig-survey_question_type_binary-hda
Open

[19.0][MIG] survey_question_type_binary#216
glitchov wants to merge 13 commits into
OCA:19.0from
acsone:19.0-mig-survey_question_type_binary-hda

Conversation

@glitchov

Copy link
Copy Markdown

Migration from 17.0

Jose Zambudio and others added 13 commits March 6, 2026 14:51
Issue: no answer was created if question not filled. If filled is was marked as Skipped
- Create a Skipped answer when the binary question is not filled
- If filled, put the file name is the Answer or the number of files in case of binary_multi
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: survey-16.0/survey-16.0-survey_question_type_binary
Translate-URL: https://translation.odoo-community.org/projects/survey-16-0/survey-16-0-survey_question_type_binary/
Currently translated at 100.0% (38 of 38 strings)

Translation: survey-16.0/survey-16.0-survey_question_type_binary
Translate-URL: https://translation.odoo-community.org/projects/survey-16-0/survey-16-0-survey_question_type_binary/it/

@FrancoMaxime FrancoMaxime 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.

LGTM: code review

layadelcarmen pushed a commit to layadelcarmen/survey that referenced this pull request Jun 17, 2026
Backport of OCA#216 (19.0 migration) adapted for 18.0:
- JS rewritten for publicWidget (Odoo 18 still uses legacy widget system)
- self.env._() translation API, Command.create(), _validate_binary convention
- Replace deprecated get_module_resource with file_path in tests

@sergiocorato sergiocorato 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.

The commits of this migration do not respect guidelines (1 commit for pre-commit autofixes and 1 for migration, check oca-port to do them automatically).

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants