server-ux icon indicating copy to clipboard operation
server-ux copied to clipboard

[16.0][ADD] confirmation_wizard

Open geomer198 opened this issue 1 year ago • 2 comments

The module lets the developer trigger a confirmation wizard during any action in Odoo. Based on data passed to the wizard and, based on user input, executes specified methods or close wizard.

geomer198 avatar Aug 27 '24 20:08 geomer198

Sorry @Bearnard21 you are not allowed to rebase.

To do so you must either have push permissions on the repository, or be a declared maintainer of all modified addons.

If you wish to adopt an addon and become it's maintainer, open a pull request to add your GitHub login to the maintainers key of its manifest.

OCA-git-bot avatar Aug 31 '24 15:08 OCA-git-bot

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). 🤖

OCA-git-bot avatar Sep 23 '24 11:09 OCA-git-bot

Hi @OCA/server-environment-maintainers, any chances to have this reviewed and merged?

ivs-cetmix avatar Nov 26 '24 21:11 ivs-cetmix

@ivs-cetmix @GabbasovDinar People might have different opinions, but mine is that such a simple tool might get a broader adoption if licensed under LGPL rather than AGPL. Many companies would simply replicate the feature without adopting it if licensed as AGPL. In a word, my opinion is simple framework tools are better licensed under LGPL. But of course the choice is yours and this won't impact for merging the contrib.

rvalyi avatar Dec 29 '24 14:12 rvalyi

@ivs-cetmix @GabbasovDinar People might have different opinions, but mine is that such a simple tool might get a broader adoption if licensed under LGPL rather than AGPL. Many companies would simply replicate the feature without adopting it if licensed as AGPL. In a word, my opinion is simple framework tools are better licensed under LGPL. But of course the choice is yours and this won't impact for merging the contrib.

Sounds reasonable, we will change the license to "LGPL-3".

ivs-cetmix avatar Dec 29 '24 14:12 ivs-cetmix

@rvalyi Could you please check again?

geomer198 avatar Feb 25 '25 12:02 geomer198

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). 🤖

OCA-git-bot avatar Feb 25 '25 13:02 OCA-git-bot

/ocabot merge nobump

rvalyi avatar Feb 25 '25 14:02 rvalyi

This PR looks fantastic, let's merge it! Prepared branch 16.0-ocabot-merge-pr-942-by-rvalyi-bump-nobump, awaiting test results.

OCA-git-bot avatar Feb 25 '25 14:02 OCA-git-bot

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

OCA-git-bot avatar Feb 25 '25 14:02 OCA-git-bot