Skip to content

Use Discard rather than Cancel #7596

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
matho-odoo wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Use Discard rather than Cancel #7596

matho-odoo wants to merge 1 commit into from

Conversation

@matho-odoo
Copy link

@matho-odoo matho-odoo commented Dec 2, 2025 •
edited
Loading

Description:

Use Discard rather than Cancel

Task: 5353199

review checklist

  • feature is organized in plugin, or UI components
  • support of duplicate sheet (deep copy)
  • in model/core: ranges are Range object, and can be adapted (adaptRanges)
  • in model/UI: ranges are strings (to show the user)
  • undo-able commands (uses this.history.update)
  • multiuser-able commands (has inverse commands and transformations where needed)
  • new/updated/removed commands are documented
  • exportable in excel
  • translations (_t("qmsdf %s", abc))
  • unit tested
  • clean commented code
  • track breaking changes
  • doc is rebuild (npm run doc)
  • status is correct in Odoo

@robodoo
Copy link
Collaborator

robodoo commented Dec 2, 2025

Pull request status dashboard

@matho-odoo matho-odoo force-pushed the master-use-discard-matho branch from 18c469d to f398ce8 Compare December 2, 2025 16:03
@matho-odoo matho-odoo changed the title [WIP] Use Discard rather than Cancel Dec 2, 2025
@matho-odoo matho-odoo force-pushed the master-use-discard-matho branch from f398ce8 to b19e6e9 Compare December 3, 2025 13:01
@matho-odoo
[IMP] Use Discard rather than Cancel
d2fefe6 
Replaced all cancel buttons with discard

Task: 5353199
@matho-odoo matho-odoo force-pushed the master-use-discard-matho branch from b19e6e9 to d2fefe6 Compare December 3, 2025 13:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@matho-odoo @robodoo