Skip to content

sandboxes: warn users that merge is dumb #4520

@brandonjackson

Description

@brandonjackson

Problem Statement

  • Sandboxes merge is a blunt instrument
  • The merge UI is not very clear about the risks of merging
Image

Solution

  • This will overwrite the selected workflows in brandon-test with the versions from foo, then delete foo. Any changes in brandon-test that conflict with foo will be lost.
  • Replace beta warning with red "This action cannot be undone"

Ideas considered / not implemented:

  • Replacing warning banner with "type name of sandbox" input. Decided against because this would probably get annoting. We want sandboxes to feel quick.
  • Keeping CLI reference. Decided against because we need to focus on complete clarity about the risk.

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

No type

Projects

Status

In review

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions