Integration support for feeless attestation #5426
Assignees
Labels
Milestone
Comments
mergify bot
pushed a commit
that referenced
this issue
Oct 31, 2020
### Description Adds reCaptcha implementation behind a feature flag. If `KOMENCI` is `true`, after clicking on `Start` on the `VerificationEducationScreen` the user will be prompted with a reCaptcha challenge that they'll need to complete before being able to verify. It also adds [SafetyNet](https://developer.android.com/training/safetynet/attestation) which is a check for Android devices. This is missing integration with KomenciKit ### Tested By changing the value of the KOMENCI flag and seeing the captcha flow (captcha code is logged) ### Related issues - Part of #5426
This was referenced Nov 2, 2020
mergify bot
pushed a commit
that referenced
this issue
Nov 3, 2020
### Description If someone uses a phone number from a sanctioned country, they get stuck with an error on the screen to input name & number and can't continue to use the app. ### Tested On the simulator with phone numbers from each of the banned countries. ### Related issues - Fixes #5661 - Part of #5426
mergify bot
pushed a commit
that referenced
this issue
Nov 6, 2020
### Description - Adds a new item to the `NavigationDrawer` that opens a popup which allows a user to invite their friends. - On the send flow, if the user is inviting someone, the 30 cent invite fee is no longer added and the transaction to send those 30 extra cents doesn't happen. Those two things will only be enabled when the `KOMENCI` feature flag is turned on (it's off right now). Note: Some of the texts are still under revision and might change. ### Other changes Added a small change to the `VerificationEducationScreen` that was missing from a previous PR. ### Tested Going through the Send flow inviting someone and seeing that no extra fee gets charged and nothing else unexpected happens. Also pressing on the Invite item on the drawer and trying to invite someone. ### Related issues - Part of #5426
mergify bot
pushed a commit
that referenced
this issue
Nov 9, 2020
### Description Removed the `name` from the invite link texts since it was using the user's names and it made no sense. ### Other changes Fixed a typo `facil` -> `fácil` ### Tested Manually ### Related issues - Part of #5426 ### Backwards compatibility N/A
|
The work on this is done for now :) |
This was referenced Jun 21, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
What's important about this?
Feeless attestation is a blocker for GA. We want to be sure the integration with the wallet is optimal.
Definition of "done"
Feeless attestation is integrated and fully functional.
What's involved in doing this work?
Open questions
The text was updated successfully, but these errors were encountered: