Skip to content

[engine] Only show IAP notice if user has not already verified front#5941

Ben requested to merge feat/verify-unique-notice-f5941 into master

Ticket(s) / Related Merge Requests

Summary of Changes

Hides in-app verification notice when verified via in-app verify, and fixes issue where if mob number is verified it will auto-hide which was out of scope but easy to fix.

Testing Considerations

You will need to manually edit data in Vitess. This is not something that will be testable outside of the sandbox env yet.

You need to go into the user_verifications table and make a new row for your users GUID, you can copy values from other rows.

  • Change status to 2 if you want your user to be verified - you should NOT see the notice.
  • Change status to 1 if you want your user to be NOT verified - you should see the notice

Note - make sure you use an account with email confirmed and no pending Superminds, else you will see the wrong respective notice.

Deployment Considerations

Standard deployment - I alerted mobile so as not to mess with their testing flow.

Regression Scope

In-app verify notice show conditions. (unreleased)

Platform Affected (web, mobile, etc)

Web

Developer Testing Completed

simplescreenrecorder-2023-04-03_15.15.36

Screenshots / Screen Recording

Does this impact

  • Localization
  • Dark/light mode
  • Guest mode

Definition of Done Checklist

  • The Acceptance Criteria has been met
  • Code is tested: Testing includes unit/spec, E2E/automated and manual testing
  • Merge requests description has been filled out
Edited by Ben

Merge request reports