web icon indicating copy to clipboard operation
web copied to clipboard

[16.0][MIG] web_m2x_options_manager : Migrate to 16.0

Open QuocDuong1306 opened this issue 2 years ago • 11 comments

This PR takes over:

  • https://github.com/OCA/web/pull/2448

QuocDuong1306 avatar Aug 11 '23 05:08 QuocDuong1306

/ocabot migration web_m2x_options_manager

pedrobaeza avatar Aug 11 '23 05:08 pedrobaeza

Thanks for your reviews, I updated

QuocDuong1306 avatar Sep 04 '23 05:09 QuocDuong1306

Can you check if the "Create & Edit Wizard" is working? I don't see any difference when is True and when is False.

moitabenfdz avatar Dec 18 '23 11:12 moitabenfdz

/ocabot merge nobump

dreispt avatar Apr 15 '24 11:04 dreispt

What a great day to merge this nice PR. Let's do it! Prepared branch 16.0-ocabot-merge-pr-2587-by-dreispt-bump-nobump, awaiting test results.

OCA-git-bot avatar Apr 15 '24 11:04 OCA-git-bot

@dreispt your merge command was aborted due to failed check(s), which you can inspect on this commit of 16.0-ocabot-merge-pr-2587-by-dreispt-bump-nobump.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

OCA-git-bot avatar Apr 15 '24 11:04 OCA-git-bot

@QuocDuong1306 could you please fix pre-commit issues? Thanks!

danielduqma avatar May 06 '24 07:05 danielduqma

@QuocDuong1306 After reviewing the previous comment in the PR where the error is located, will changes be added to continue the migration?

aliciagaarzo avatar Jul 11 '24 15:07 aliciagaarzo

Hi, sorry for the delay, I will update the PR asap

QuocDuong1306 avatar Jul 15 '24 01:07 QuocDuong1306

Hello guys, the PR is updated

QuocDuong1306 avatar Aug 06 '24 07:08 QuocDuong1306

However, when dealing with the res.partner model, I used the same "Force remove" configuration for the team_id and user_id fields. Unfortunately, while this configuration works for team_id, it does not effectively transfer to the user_id field, meaning that editing and creation in this field are still allowed.

Hello @aliciagaarzo , the issue is fixed, could you have a look?

Screencast from 2024-08-06 14-18-37.webm

QuocDuong1306 avatar Aug 06 '24 07:08 QuocDuong1306

Trusting in reviewers

/ocabot merge nobump

pedrobaeza avatar Nov 13 '24 15:11 pedrobaeza

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

OCA-git-bot avatar Nov 13 '24 15:11 OCA-git-bot

Congratulations, your PR was merged at b591539403f5b228ef13fe0f00f041cea34bea99. Thanks a lot for contributing to OCA. ❤️

OCA-git-bot avatar Nov 13 '24 16:11 OCA-git-bot