Skip to content
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

Safe Mode Activated Banner / Modal is broken #9614

Open
AashikP opened this issue Oct 22, 2024 · 0 comments
Open

Safe Mode Activated Banner / Modal is broken #9614

AashikP opened this issue Oct 22, 2024 · 0 comments
Labels
focus: account lifecycle priority: medium The issue/PR is medium priority—non-critical functionality loss, minimal effect on usability type: bug The issue is a confirmed bug.

Comments

@AashikP
Copy link

AashikP commented Oct 22, 2024 •
edited by anu-rock
Loading

Describe the bug

Safe mode banner is broken. The original options Transfer Connection and Create a new connection is replaced by Create a new connection and Stay in safe mode, and the description provided for both options are same.

Expected As per WooPayments Safe Mode - Documentation

Image

Current:

Image

To Reproduce

  1. Have a live account
  2. Create a staging site, which will have the WooPayments account set to safe mode
  3. The original options Transfer Connection and Create a new connection is replaced by Create a new connection and Stay in safe mode
  4. Find that the description provided for both options are same

Actual behavior

Safe mode activated banner is confusing,

Expected behavior

Bring back the previous banner

Additional context

Slack: p1729589090376169-slack-C7U3Y3VMY
Merchant report: 8838935-zd-a8c

Dev notes

See this comment from @htdat about a potential solution p1731901709055269/1729589090.376169-slack-C7U3Y3VMY.
@AashikP AashikP added the type: bug The issue is a confirmed bug. label Oct 22, 2024
@aheckler aheckler added the needs triage Manually put issue into triage process. label Oct 23, 2024
@bborman22 bborman22 added focus: account lifecycle and removed needs triage Manually put issue into triage process. labels Oct 23, 2024
@anu-rock anu-rock added priority: low The issue/PR is low priority—not many people are affected or there’s a workaround, etc. priority: medium The issue/PR is medium priority—non-critical functionality loss, minimal effect on usability and removed priority: low The issue/PR is low priority—not many people are affected or there’s a workaround, etc. labels Nov 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
focus: account lifecycle priority: medium The issue/PR is medium priority—non-critical functionality loss, minimal effect on usability type: bug The issue is a confirmed bug.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@anu-rock @aheckler @AashikP @bborman22