Skip to content

Update copy in account recovery settings global alert

📖 What does this MR do?

Related to #297366 (closed)

Updates the copy in the account recovery settings alert let the user know they should ensure their 2FA settings are up to date. Previously there was some confusion about what settings should be checked. This MR also takes the opportunity to refactor this alert to use the app/views/shared/_global_alert.html.haml HAML partial which helps improve consistency of global alerts.

💻 Local testing

  1. Download and apply this patch: test-alert.patch
    • git apply <path to patch>
  2. Delete the UserCallout record for this alert
    • bin/rails console
    • UserCallout.where({ feature_name: 12 }).destroy_all
  3. Reload the page

📷 Screenshots

View Before After
Desktop Screen_Shot_2021-07-28_at_1.07.30_PM Screen_Shot_2021-08-02_at_9.19.51_AM
Mobile Screen_Shot_2021-07-28_at_1.08.03_PM Screen_Shot_2021-08-02_at_9.21.19_AM

🚦 Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Edited by Peter Hegman

Merge request reports

Loading