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

Re-verification Failure Should Display "Not Verified Yet" Instead of "Expired" #2663

Closed
erichfi opened this issue Jul 2, 2024 · 0 comments · Fixed by #2670
Closed

Re-verification Failure Should Display "Not Verified Yet" Instead of "Expired" #2663

erichfi opened this issue Jul 2, 2024 · 0 comments · Fixed by #2670
Assignees

Comments

@erichfi
Copy link
Collaborator

erichfi commented Jul 2, 2024

User Story:

As a user,
I want the system to show "Not Verified Yet" instead of "Expired" when re-verification fails,
So that I can understand that my stamp is in a non-verified state rather than incorrectly assuming it has expired.

Acceptance Criteria

GIVEN a user with an expired stamp,
WHEN the user attempts re-verification and fails,
THEN the system should display "Not Verified Yet" instead of "Expired" on the stamp.

Product & Design Links:

Tech Details:

  • Modify the logic handling re-verification outcomes to distinguish between expired and not yet verified states.
  • Update UI components to reflect the "Not Verified Yet" status appropriately.
  • Ensure consistency in the messaging across different parts of the application where the stamp status is displayed.

Open Questions:

Notes/Assumptions:

  • Assumes that the backend can distinguish between an expired stamp and a failed re-verification attempt.
  • Assumes that this change applies to all types of stamps within the system.
@erichfi erichfi added this to Passport Jul 2, 2024
@erichfi erichfi moved this to Prioritized in Passport Jul 2, 2024
@tim-schultz tim-schultz self-assigned this Jul 2, 2024
@tim-schultz tim-schultz moved this from Prioritized to In Progress (WIP) in Passport Jul 2, 2024
@tim-schultz tim-schultz moved this from In Progress (WIP) to Code Complete in Passport Jul 3, 2024
@tim-schultz tim-schultz moved this from Code Complete to In Progress (WIP) in Passport Jul 8, 2024
@tim-schultz tim-schultz moved this from In Progress (WIP) to Code Complete in Passport Jul 8, 2024
@tim-schultz tim-schultz moved this from Code Complete to Product/UX Review in Passport Jul 8, 2024
@erichfi erichfi moved this from Product/UX Review to Ready to Deploy in Passport Jul 9, 2024
@erichfi erichfi moved this from Ready to Deploy to Done in Passport Jul 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants