Skip to content
Snippets Groups Projects

Add separator + confirm modals in the process status actions

Merged ml bonhomme requested to merge danger-danger into master
All threads resolved!

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Erwan Rouchet
  • This menu would be better organized as:

    • View element
    • Process configuration
    • Workers activity
    • Create template

    • Retry process
    • Stop process

    This sorts the actions from the least to most dangerous.

  • ml bonhomme resolved all threads

    resolved all threads

  • ml bonhomme added 1 commit

    added 1 commit

    • 15fdcc64 - Change menu items order + throw unsupported action error

    Compare with previous version

  • Erwan Rouchet
  • Erwan Rouchet
  • Erwan Rouchet
  • ml bonhomme added 1 commit

    added 1 commit

    Compare with previous version

  • ml bonhomme resolved all threads

    resolved all threads

  • Erwan Rouchet added 4 commits

    added 4 commits

    • 4e91cded - 1 commit from branch master
    • 83cf5f80 - Add separator + confirm modals in the process status actions
    • 96682442 - Change menu items order + throw unsupported action error
    • 09540559 - reorder + dropdown-divider

    Compare with previous version

  • Erwan Rouchet enabled an automatic merge when the pipeline for 09540559 succeeds

    enabled an automatic merge when the pipeline for 09540559 succeeds

  • Please register or sign in to reply
    Loading