-
Notifications
You must be signed in to change notification settings - Fork 80
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
feat(onboarding): add terms and conditions colloquial variant #5293
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #5293 +/- ##
=======================================
Coverage 86.01% 86.02%
=======================================
Files 746 746
Lines 30520 30537 +17
Branches 5291 5294 +3
=======================================
+ Hits 26252 26268 +16
- Misses 4036 4037 +1
Partials 232 232
... and 1 file with indirect coverage changes Continue to review full report in Codecov by Sentry.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Maybe we should change this to a functional component, but that's outside of the scope of the ticket.
…-inc#5293) ### Description Implements the colliquial terms variant for the terms and conditions experiment. - [Figma](https://www.figma.com/file/mthUlChSbWXXxLe2AV13Wd/Onboarding-2024?type=design&node-id=2071-2370&mode=design&t=Z20oNRHCqe8eUHLs-0) - [statsig experiment](https://console.statsig.com/4plizaPmWwPL21ASV4QAO0/experiments/onboarding_terms_and_conditions/setup) ### Test plan | | Control | Colloquial | |--------|--------|--------| | iOS | <video src="https://github.com/valora-inc/wallet/assets/5062591/e53f3c26-caee-4a0c-b63b-1e8bf48dcf0f" /> | <video src="https://github.com/valora-inc/wallet/assets/5062591/b62b96e0-de82-4d3b-98f2-28fd73d509af" /> | | Android | <video src="https://github.com/valora-inc/wallet/assets/5062591/bf56c776-4efa-4462-9408-4e28277afe74" /> | <video src="https://github.com/valora-inc/wallet/assets/5062591/f064397b-a830-42b0-950a-e6398a8e6b15" /> | ### Related issues - Part of ACT-1162 ### Backwards compatibility N/A ### Network scalability N/A
Description
Implements the colliquial terms variant for the terms and conditions experiment.
Test plan
ios-tc-control.mp4
ios-tc-colloquial.mp4
android-tc-control.webm
android-tc-colloquial.webm
Related issues
Backwards compatibility
N/A
Network scalability
N/A