Skip to content

[19.0][MIG] web_widget_dropdown_dynamic: Migration to 19.0 - #3617

Open
idris-domatix wants to merge 23 commits into
OCA:19.0from
idris-domatix:19.0-mig-web_widget_dropdown_dynamic
Open

idris-domatix wants to merge 23 commits into
OCA:19.0from
idris-domatix:19.0-mig-web_widget_dropdown_dynamic

Conversation

@idris-domatix

Copy link
Copy Markdown

Migration of web_widget_dropdown_dynamic to 19.0

Standard migration. Module history ported from 18.0.

Migration changes:

  • Manifest version bumped to 19.0.1.0.0.
  • JS widget adapted to the 19.0 web framework (field_dynamic_dropdown.esm.js): the field renders as .o_select_menu (OWL), no OwlError.
  • models/ir_filters.py adjusted for 19.0.

Validation (local, without demo, mirroring CI):

  • pre-commit green (0 errors).
  • Install with --test-enable --without-demo=all: clean, 0 failed, 0 error, module loads without traceback.
  • Browser smoke: field renders, dropdown opens options and persists the integer selection.

No Python tests in this module (only a JS QUnit suite).

[UPD] Update web_widget_dropdown_dynamic.pot

[UPD] Update web_widget_dropdown_dynamic_example.pot

[UPD] README.rst

[ADD] icon.png

[UPD] README.rst
@OCA-git-bot OCA-git-bot added mod:web_widget_dropdown_dynamic Module web_widget_dropdown_dynamic series:19.0 labels Jul 22, 2026
Comment on lines +8 to +9
El ancla y el contexto se actualizan en consecuencia; ahora
depending_on llega como lista de ids. -->

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

could you do in english everywhere? In fact, you should delete migration comments, as they are not useful once PR is merged.

@idris-domatix

Copy link
Copy Markdown
Author

Will switch the remaining comments to English and drop the migration comments in the next push.

@idris-domatix
idris-domatix force-pushed the 19.0-mig-web_widget_dropdown_dynamic branch from b3f6e80 to 9d1acfb Compare September 4, 2026 07:03
@idris-domatix

Copy link
Copy Markdown
Author

Done: comments in the module are now in English and the migration comments have been dropped.

@idris-domatix
idris-domatix force-pushed the 19.0-mig-web_widget_dropdown_dynamic branch from 9d1acfb to 517bcc5 Compare September 4, 2026 07:24

@Andrii9090-tecnativa Andrii9090-tecnativa 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.


await editInput(target, ".o_field_widget[name='change_field'] input", "step-1");
assert.containsN(target, "option", 2);
assert.containsOnce(target, "option[value='\"value\"']");

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Can you review this test? In v19, the elements option no longer exists, and SelectMenu is rendered differently.

Image

@idris-domatix
idris-domatix force-pushed the 19.0-mig-web_widget_dropdown_dynamic branch from 517bcc5 to c820d87 Compare September 10, 2026 07:45
@idris-domatix

Copy link
Copy Markdown
Author

Thanks for the review.

  • Commits squashed: the branch is now a single commit [19.0][MIG] web_widget_dropdown_dynamic: Migration to 19.0, rebased on top of the latest 19.0 (the bot/Weblate commits are gone).
  • JS test reviewed: the test was still written for the v18 QUnit helpers and relied on <option> elements, which no longer exist in v19. I migrated it to Hoot (web.assets_unit_tests) and updated it to the v19 SelectMenu DOM (.o_select_menu_item), following the core selection_field tests.

Verified locally on 19.0: 4/4 Hoot tests and 4/4 Python tests pass.

@Andrii9090-tecnativa

Copy link
Copy Markdown

Thanks for the review.

  • Commits squashed: the branch is now a single commit [19.0][MIG] web_widget_dropdown_dynamic: Migration to 19.0, rebased on top of the latest 19.0 (the bot/Weblate commits are gone).
  • JS test reviewed: the test was still written for the v18 QUnit helpers and relied on <option> elements, which no longer exist in v19. I migrated it to Hoot (web.assets_unit_tests) and updated it to the v19 SelectMenu DOM (.o_select_menu_item), following the core selection_field tests.

Verified locally on 19.0: 4/4 Hoot tests and 4/4 Python tests pass.

You need to squash the administrative commits. Otherwise, we will lose the entire commit history.
https://github.com/OCA/maintainer-tools/wiki/Merge-commits-in-pull-requests#mergesquash-the-commits-generated-by-bots-or-weblate

@pedrobaeza

Copy link
Copy Markdown
Member

Yes, you have to squash some commits (administrative and translations one if proceed), not everyone.

Mikhail Lapin and others added 4 commits September 10, 2026 19:29
Change the License to LGPL-3 in order to be able to use this module with other modules with non-AGPL-3 licenses.

Task:4743
Currently translated at 100.0% (3 of 3 strings)

Translation: web-18.0/web-18.0-web_widget_dropdown_dynamic
Translate-URL: https://translation.odoo-community.org/projects/web-18-0/web-18-0-web_widget_dropdown_dynamic/es/
@idris-domatix
idris-domatix force-pushed the 19.0-mig-web_widget_dropdown_dynamic branch from c820d87 to 784fe15 Compare September 10, 2026 17:30
@idris-domatix

Copy link
Copy Markdown
Author

Thanks @Andrii9090-tecnativa and @pedrobaeza. I had squashed everything into a single commit, which was wrong.

I have restored the module commit history and rebased to fixup only the administrative commits:

  • [BOT] post-merge updates, [UPD] Update *.pot and Update translation files are now squashed into the contributor commit that generated them.
  • Weblate translation commits from the same author and language have been merged together.
  • The contributor commits ([ADD], [MIG] 13.0 → 19.0, [IMP]...) are preserved, and the [19.0][MIG] commit now also includes the JS test migration to Hoot.

The module content is unchanged (4/4 Hoot tests and 4/4 Python tests pass).

@pedrobaeza

Copy link
Copy Markdown
Member

/ocabot migration web_widget_dropdown_dynamic

@OCA-git-bot OCA-git-bot added this to the 19.0 milestone Sep 11, 2026
@OCA-git-bot OCA-git-bot mentioned this pull request Sep 11, 2026
28 of 46 tasks
@Andrii9090-tecnativa

Copy link
Copy Markdown

@idris-domatix can you check the tests?

@idris-domatix
idris-domatix force-pushed the 19.0-mig-web_widget_dropdown_dynamic branch from 784fe15 to 6097035 Compare September 15, 2026 09:26
@Andrii9090-tecnativa

Copy link
Copy Markdown

@idris-domatix can you repeat please?

@idris-domatix
idris-domatix force-pushed the 19.0-mig-web_widget_dropdown_dynamic branch from 6097035 to bae84e7 Compare October 7, 2026 07:23
@idris-domatix

Copy link
Copy Markdown
Author

Thanks for the review.

  • Removed the migration comments; the module now keeps only the code.
  • Migration commit subject fixed to [MIG] web_widget_dropdown_dynamic: Migration to 19.0.
  • The tests were already rewritten for the v19 SelectMenu (they now use .o_select_menu_item).
  • The previous CI failures were infrastructure-only ("The job was not started because it repeatedly failed to be acquired"), so this run should be green.

About the squash request: there are no bot [UPD] commits in the diff. The only same-author/same-language Weblate duplicate (Italian, mymage) is separated by the [MIG] ... 17.0 commit, which also edits it.po, so merging them would conflict; per the wiki those are left as is.

@MiquelRForgeFlow @Andrii9090-tecnativa could you take another look when you have a moment?

@OCA-git-bot

Copy link
Copy Markdown
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

@pedrobaeza

Copy link
Copy Markdown
Member

/ocabot merge nobump

@OCA-git-bot

Copy link
Copy Markdown
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 19.0-ocabot-merge-pr-3617-by-pedrobaeza-bump-nobump, awaiting test results.

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.