[FW][IMP] stock, mrp: Transfer,MO cancel warning #164911
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
It's too easy to cancel a picking or a manufacturing by mistake. In the opposite it's hard to reverse the error since there is no reset to draft or any easy way to remove the cancel state. The best solution would be to have an easy way to fix the issue but until a proper solution a simple warning message could be useful X-original-commit: b4a395a
|
|
|
This PR targets saas-17.1 and is part of the forward-port chain. Further PRs will be created up to master. More info at https://github.com/odoo/odoo/wiki/Mergebot#forward-port |
|
@fd-oerp @amoyaux the next pull request (#164934) is in conflict. You can merge the chain up to here by saying
More info at https://github.com/odoo/odoo/wiki/Mergebot#forward-port |
|
@fw-bot r+ |
robodoo
pushed a commit
that referenced
this pull request
May 29, 2024
It's too easy to cancel a picking or a manufacturing by mistake. In the opposite it's hard to reverse the error since there is no reset to draft or any easy way to remove the cancel state. The best solution would be to have an easy way to fix the issue but until a proper solution a simple warning message could be useful closes #164911 X-original-commit: b4a395a Signed-off-by: Arnold Moyaux (arm) <arm@odoo.com> Signed-off-by: Tiffany Chang (tic) <tic@odoo.com>
robodoo
pushed a commit
that referenced
this pull request
May 29, 2024
It's too easy to cancel a picking or a manufacturing by mistake. In the opposite it's hard to reverse the error since there is no reset to draft or any easy way to remove the cancel state. The best solution would be to have an easy way to fix the issue but until a proper solution a simple warning message could be useful closes #164911 X-original-commit: b4a395a Signed-off-by: Arnold Moyaux (arm) <arm@odoo.com> Signed-off-by: Tiffany Chang (tic) <tic@odoo.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description of the issue/feature this PR addresses:
This PR aims to address the issue where the cancel buttons (on transfer and MO) are prone to accidental presses due to their proximity. Currently, pressing the cancel button immediately cancels the transfer or MO.
Current behavior before PR:
Pressing the cancel button results in immediate cancellation of the transfer or MO.
Desired behavior after PR is merged:
After merging this PR, Odoo will prompt for confirmation when the cancel button on a transfer or MO is pressed. This will help prevent accidental cancellations. Additionally, the confirmation warning should always be present for both cancel buttons on the MO to ensure consistency.
See below attached for a confirmation message:
On transfer,
On MO,
I confirm I have signed the CLA and read the PR guidelines at submit-pr
Forward-Port-Of: #160609