-
Notifications
You must be signed in to change notification settings - Fork 1
Jameson/CT-1815 #247
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
Jameson/CT-1815 #247
Conversation
…e back button visibility
…ber and Search components
…ationHeader, Search, and VerifyExistingMember components
|
looks good! |
|
Is the requirement to show the back button only on the |
|
yes it only needs to be available on the those first screens so the user can go back a step into the mobile app where they entered the flow from; i'm not super we ever pass in institutionID or memberID, but theoretically those should have back buttons that go to the previous mobile screen (mobile will control that) |
|
I didn't consider the returning user flow where we start mid widget flow. Thanks a ton for catching that, I have it all updated to be specific to our first step of SEARCH or VERIFY_EXISTING_MEMBER. Ready for another review. |
…tton visibility logic
…pectation for GO_BACK_CREDENTIALS action
|
Testing Connect with back button enabled
|
|
🎉 This PR is included in version 2.14.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
https://mxcom.atlassian.net/browse/CT-1815
Changes
VerifyExistingMemberandSearchsteps.Testing instructions
*This change is high impact. Manually test this throughly with a variety of settings/conditions.
With mobile back button

Without mobile back button
