Make reverification comms more clear by skipping overview screen and improving action screen#4375
Make reverification comms more clear by skipping overview screen and improving action screen#4375Copilot wants to merge 5 commits into
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub. 3 Skipped Deployments
|
Agent-Logs-Url: https://github.com/GoodDollar/GoodDAPP/sessions/93d0604a-f740-457e-977a-dcaa0a37309c Co-authored-by: sirpy <671095+sirpy@users.noreply.github.com>
Agent-Logs-Url: https://github.com/GoodDollar/GoodDAPP/sessions/93d0604a-f740-457e-977a-dcaa0a37309c Co-authored-by: sirpy <671095+sirpy@users.noreply.github.com>
|
@copilot can you explain what the component IntroReVerification is for? is it used? |
…ckboxes Agent-Logs-Url: https://github.com/GoodDollar/GoodDAPP/sessions/0b65301a-ba77-4e2b-aca6-fad6133f9756 Co-authored-by: sirpy <671095+sirpy@users.noreply.github.com>
Since it's a separate code path for reverifying users, I've updated it in commit |
|
@copilot can you test it locally that it works and produce a screenshot? |
Ran it locally with 1. Overview screen (new/first-time users only — reverifying users skip this entirely and land directly on screen 3): 2. Action screen — checkboxes unchecked, "Verify Me" disabled: 3. Action screen — both checkboxes checked, "Verify Me" enabled: Key behaviours confirmed:
|
Agent-Logs-Url: https://github.com/GoodDollar/GoodDAPP/sessions/8988dca1-e107-46a0-a58b-d0528cb21919 Co-authored-by: sirpy <671095+sirpy@users.noreply.github.com>



Reverifying users were shown the same "Verify your unique identity" overview screen as first-time users, which was confusing. The fix routes them directly to the action screen with contextually appropriate content.
Changes
Skip overview for reverifying users:
IntroFVFlowinitializesshowActionScreentoisReverify, so returning users land directly onIntroFVFlowActioninstead of seeingIntroFVFlowOverviewWallet linked box on reverification action screen: When
isReverify && walletAddress, the action screen now renders the wallet block (previously only on overview). Shows "Last verified:<date>" instead of "Valid until"Age checkbox for all users: The "I am 18+ and I'm verifying my own identity" checkbox was previously gated to
!isReverify. It's now shown unconditionally, and the "Verify Me"/"Continue" button requires both checkboxes for all userslastVerifiedprop: Derived fromexpiryDate?.lastAuthenticated(BigNumber unix timestamp) viamoment.unix()inIntroScreen, threaded down to bothIntroFVFlowActionandIntroReVerificationIntroReVerificationupdated (non-FV-flow reverification path): The legacy reverification screen used in the regular GoodDAPP flow (whenisFVFlow === false) now also receives the wallet linked box (with "Last verified" date), the age confirmation checkbox, and the GoodDollar consent checkbox. The "Continue" button is gated behind both checkboxes.