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.

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

Fix Missing Word in Identity Modal #1674

Merged
merged 3 commits into from
Nov 3, 2022
Merged

Fix Missing Word in Identity Modal #1674

merged 3 commits into from
Nov 3, 2022

Conversation

RobertKeyser
Copy link
Contributor

@RobertKeyser RobertKeyser commented Nov 2, 2022

Originally the description on the modal for ID verification states

We have sent a verification code to your email address. Please check your email, then return to this window and the code below.

I changed it to add the word enter

We have sent a verification code to your email address. Please check your email, then return to this window and the enter code below.

Code Changes

  • added missing word to modal text
  • updated changelog

Steps to Confirm

  • list any manual steps taken to confirm the changes

Pre-Merge Checklist

  • All CI Pipelines Succeeded
  • Documentation Updated:
    • documentation complete, or draft/outline provided (tag docs-team to complete/review on this branch)
    • documentation issue created (tag docs-team to complete issue separately)
  • Issue Requirements are Met
  • Relevant Follow-Up Issues Created
  • Update CHANGELOG.md

Description Of Changes

Should be a visual-only change.

@RobertKeyser RobertKeyser marked this pull request as ready for review November 2, 2022 21:46
@RobertKeyser RobertKeyser added the UI Work related to UI label Nov 2, 2022
@RobertKeyser RobertKeyser self-assigned this Nov 2, 2022
@sanders41 sanders41 merged commit fa26dbf into main Nov 3, 2022
@sanders41 sanders41 deleted the rk-verification-modal branch November 3, 2022 11:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
UI Work related to UI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants