Skip to content

Update copy in recovery settings alert

📖 What does this MR do?

Related to #297366 (closed)

In !66693 (merged) we updated the copy in this alert, but we received some feedback that the copy was still confusing. Specifically it was confusing that the alert said We recommend you ensure two-factor authentication is enabled for users that already had two-factor authentication enabled. The original intent of #297366 (closed) was to remind users to save their recovery codes so we adjusted to the copy to be more explicit about this. We also now only show this alert to users that have two-factor authentication enabled.

💻 How to setup and validate locally

  1. Enable two-factor authentication
    1. Navigate to /-/profile/account
    2. Click "Enable two-factor authentication"
    3. Open up 1Password
    4. Click + icon and then "Login"
    5. Create a new "One-Time Password" section
      • Screen_Shot_2020-12-03_at_11.33.49_AM
    6. Click the QR icon left of the dropdown that was just used
    7. Drag the QR window over the QR code
    8. Paste the OTP in the "Pin code" field and click "Register with two-factor app"
  2. Download and apply this patch: local-testing.patch
    • git apply <path to patch>
  3. Delete the UserCallout record for this alert
    • bin/rails console
    • UserCallout.where({ feature_name: 12 }).destroy_all
  4. Reload the page

📷 Screenshots

Before After
Screen_Shot_2021-08-12_at_9.55.55_AM Screen_Shot_2021-08-12_at_9.50.48_AM

🚦 Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Edited by Peter Hegman

Merge request reports

Loading