Skip to content

Conversation

@sreichel
Copy link
Contributor

@sreichel sreichel commented Dec 16, 2022

Description (*)

Added confirm dialog to some massactions.

I did not change it for every massaction, but added a default check by massaction id/name.

    protected static $needsConfirm = [
        self::CANCEL_ORDER,
        self::HOLD_ORDER,
        self::UNHOLD_ORDER,
        self::DELETE,
        self::REMOVE
    ];

Related Pull Requests

Fixed Issues (if relevant)

  1. Fixes Some mass actions in Orders grid don't have a confirmation dialog #2323

Manual testing scenarios (*)

  1. ...
  2. ...

Questions or comments

Contribution checklist (*)

  • Pull request has a meaningful description of its purpose
  • All commits are accompanied by meaningful commit messages
  • All automated tests passed successfully (all builds are green)
  • Add yourself to contributors list

@github-actions github-actions bot added Component: Adminhtml Relates to Mage_Adminhtml Component: Index Relates to Mage_Index labels Dec 16, 2022
@sreichel sreichel changed the title Add confirm to critical massactions Add confirm dialog to critical massactions Dec 17, 2022
@fballiano
Copy link
Contributor

quickly tested, seems to be working fine

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Component: Adminhtml Relates to Mage_Adminhtml Component: Index Relates to Mage_Index phpstan

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Some mass actions in Orders grid don't have a confirmation dialog

3 participants