Skip to content
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

[Wallet] Changes to Simplex cash in #5855

Merged
merged 10 commits into from
Nov 13, 2020
Merged

[Wallet] Changes to Simplex cash in #5855

merged 10 commits into from
Nov 13, 2020

Conversation

gnardini
Copy link
Contributor

Description

Implement the changes discussed in this thread: https://celo-org.slack.com/archives/CL7BVQPHB/p1605212144230400

  • Open Simplex webpage instead of using a webview.
  • Selection: right now, seems like the selection for cUSD/CELO is dependent on the method selected vs the country (eg CELO should always be selectable, but then only provide the option for ‘cryptocurrency exchange’
  • Debit Card copy change-- update “Debit, Credit, or Bank Account” to “Debit Card or Bank Account”

Tested

Manually

Copy link
Contributor

@jeanregisser jeanregisser left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome! thanks 💯

Seeing the CI errors, I guess we need to remove the Screens.Simplex reference.

@gnardini gnardini added the automerge Have PR merge automatically when checks pass label Nov 12, 2020
@annakaz
Copy link
Contributor

annakaz commented Nov 13, 2020

Still working on getting iOS tests passing

@mergify mergify bot merged commit 8f70672 into master Nov 13, 2020
@mergify mergify bot deleted the cico-changes branch November 13, 2020 03:00
i1skn pushed a commit that referenced this pull request Nov 13, 2020
### Description

Implement the changes discussed in this thread: https://celo-org.slack.com/archives/CL7BVQPHB/p1605212144230400
- Open Simplex webpage instead of using a webview.
- Selection: right now, seems like the selection for cUSD/CELO is dependent on the method selected vs the country (eg CELO should always be selectable, but then only provide the option for ‘cryptocurrency exchange’
- Debit Card copy change-- update “Debit, Credit, or Bank Account” to “Debit Card or Bank Account”

### Tested

Manually
@Lss-Ankit
Copy link

Hi @tarikbellamine i Have verified this issue latest test flight build v1.5.5 (34) and Android play store internal build v1.5.5 (1004294323) and observe the following

  • User redirected to web page ( Valoraapp.com ) when user tap on next button after selecting "Celo Dollar" & Debit crd or bank account" or "cryptocurrency exchange"
  • User redirected to exchanges page when user tap on next button after selecting "Celo" & cryptocureency echange"

Devices: iPhone 6+ (12.4.5) , Vivo v11 pro (10)

Can you please let us know if we need to test anything else in this task

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge Have PR merge automatically when checks pass wallet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants