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

[Android] Add Gemini user wallet support #17408

Closed
Miyayes opened this issue Aug 9, 2021 · 22 comments · Fixed by brave/brave-core#12632
Closed

[Android] Add Gemini user wallet support #17408

Miyayes opened this issue Aug 9, 2021 · 22 comments · Fixed by brave/brave-core#12632

Comments

@Miyayes
Copy link
Collaborator

Miyayes commented Aug 9, 2021

Description

Add support for Gemini user wallets on Android.

Test plan

https://github.com/brave/internal/issues/865

@Miyayes Miyayes added feature/rewards OS/Android Fixes related to Android browser functionality parity labels Aug 9, 2021
@Miyayes Miyayes added the priority/P3 The next thing for us to work on. It'll ride the trains. label Oct 20, 2021
@prashasth-nair
Copy link

Can you please tell will the next update will add support for Gemini in android?

@aman-m1
Copy link

aman-m1 commented Nov 6, 2021

I am also patiently waiting for this. Gemini was available for creators in August-2020 and it became available in desktop a year later in August-2021. At that time a couple of months was estimate guess of some moderators on reddit and community page for it to be available on Android pending rewrite of rewards UI, which is not yet released. I hope it doesn't get dragged into another year.

@prashasth-nair
Copy link

Can you tell when will this feature will be added?

@aman-m1
Copy link

aman-m1 commented Dec 18, 2021

New rewards UI on Android has landed in Nightly v1.35.44 and it looks good.
Screenshot_2021-12-18-10-11-57-12_82cebf612ef3f9a8762672c971adaed4
Bitflyer has also been added a while ago- #18439
1 Feb 2022 is due date for stable release of v1.35.xx

@Miyayes
Copy link
Collaborator Author

Miyayes commented Dec 23, 2021

@aman-m1, there's an even newer UI now that looks even better. Still some polishing to do #20207, but is even further improved :)

@prashasth-nair As soon as we can after v1.35.

@aman-m1
Copy link

aman-m1 commented Dec 23, 2021

@Miyayes Looking forward to it.

Also I would point out one thing, the UI card on brave://rewards doesn't fit in perfectly. It would be good if we don't have to swipe left and right to get its full view. Here is a video-

Record_2021-12-23-23-09-11.mp4

@Miyayes
Copy link
Collaborator Author

Miyayes commented Dec 30, 2021

@Miyayes Looking forward to it.

Also I would point out one thing, the UI card on brave://rewards doesn't fit in perfectly. It would be good if we don't have to swipe left and right to get its full view. Here is a video-

Record_2021-12-23-23-09-11.mp4

I think this will be solved with #20173

@thevirajshelke
Copy link

So is this feature of supporting Gemini wallet on Android in progress or it's already out in the Nightly releases?

Also, when can we expect a stable release with this feature?

What users expect is that all of the BAT coins generated should somehow go to the same wallet Uphold, Gemini, etc. As of now if someone uses Gemini on the desktop, laptop & other device is Android we can't save BAT in a single place! So this feature is very important from the user experience perspective!

Hoping to see this asap in the subsequent releases! Kudos to Brave Team! Keep it up! :)

@Miyayes
Copy link
Collaborator Author

Miyayes commented Jan 6, 2022

@thevirajshelke, it is not yet in Nightly. It'll start development quite soon; we are about to land all the foundation work in 1.35, so I think we can quickly sprint toward Gemini support after 1.35.

@thevirajshelke
Copy link

thevirajshelke commented Jan 8, 2022

So right now the latest stable version rolled out is 1.33.106 and we can expect this feature in versions after 1.35 which sounds good & quick @Miyayes. That update will surely improve the user experience a lot since we all will have a common wallet (place) to dump all out BAT from any device! :-) Looking forward to that update! Thanks a lot, @Miyayes & brave team!

@CovertLeopard
Copy link

If you need additional testers for Gemini wallet support in android, let me know. I've been patiently waiting for this feature and have no problems beta testing it on my Pixel 6 Pro if asked. @Miyayes

@Miyayes Miyayes added priority/P2 A bad problem. We might uplift this to the next planned release. and removed priority/P3 The next thing for us to work on. It'll ride the trains. labels Feb 3, 2022
@newuser-web
Copy link

@Miyayes @emerick Which version of nightly are you targeting?

@aman-m1
Copy link

aman-m1 commented Mar 7, 2022

As per bat-chriscat in v1.38.x which is the current version in Nightly link.

@varma61923
Copy link

when can we expect gemini wallet support for android ?

@aman-m1
Copy link

aman-m1 commented Mar 16, 2022

Amazing. Can we uplift it also?

@Miyayes
Copy link
Collaborator Author

Miyayes commented Mar 16, 2022

@aman-m1 We are keeping this in 1.38.x and will not be uplifting it. Thanks.

@deeppandya deeppandya added this to the 1.38.x - Nightly milestone Mar 18, 2022
@Saoiray
Copy link

Saoiray commented Mar 19, 2022

Well, glad to see it's "Done" but I wish we'd have info on when to expect 1.38 to be released. This and the removal of the device limit are the two more frequent things posted on Brave Community. Anyway, thanks for keeping us updated @Miyayes and we're definitely hoping to see it roll out soon.

@Zainzzkk
Copy link

Zainzzkk commented Mar 29, 2022

^ I agree with above. It is "Done" for nightly but not for production version. Not everyone is comfortable using nightly and technically it is not done yet for the live full brave android version. Would be nice to know when it will be available for android users not on nightly or when to expect certain big releases

@LaurenWags
Copy link
Member

@Carlancer this is moving through our channels, it is currently available on Nightly (1.39.x at the moment) and Beta (1.38.x at the moment).

Once 1.38.x hits release channel (anticipated to be on or around April 26 per https://github.com/brave/brave-browser/wiki/Brave-Release-Schedule#release-channel-dates), then this feature will be available on the release version.

Note - the dates on https://github.com/brave/brave-browser/wiki/Brave-Release-Schedule#release-channel-dates are not set in stone and may have a bit of variation. We try to release as close to them as possible.

@LaurenWags LaurenWags added the QA/In-Progress Indicates that QA is currently in progress for that particular issue label Apr 8, 2022
@LaurenWags
Copy link
Member

LaurenWags commented Apr 8, 2022

Verified on Google Pixel 3 XL with Android 12 running 1.38.83 / 1.38.93

Verification notes can be found under https://github.com/brave/internal/issues/865#issuecomment-1092958660


Verified on Samsung J7 Neo with Android 7 running 1.38.93 / 1.38.104

Verification notes can be found under https://github.com/brave/internal/issues/865#issuecomment-1104384735

@LaurenWags LaurenWags added QA Pass - Android ARM and removed QA/In-Progress Indicates that QA is currently in progress for that particular issue labels Apr 21, 2022
@Uni-verse
Copy link
Contributor

Uni-verse commented Apr 25, 2022

Verification Completed on Samsung Tab S7 using

Brave	1.38.106 Chromium: 101.0.4951.41 (Official Build) (64-bit) 
Revision	93c720db8323b3ec10d056025ab95c23a31997c9-refs/branch-heads/4951@{#904}
OS	Android 12; Build/SP1A.210812.016

PASS - Gemini available in non-Japan region

Screen Shot 2022-04-25 at 1 33 15 PM

PASS - Gemini not available in Japan region

Screen Shot 2022-04-25 at 3 39 44 PM

PASS - Successfully link Gemini

Screen Shot 2022-04-25 at 2 49 06 PM

PASS - Attempt to link non-KYC'd Gemini account

  1. Clean profile, non-Japan region, enable staging env and relaunch as necessary
  2. Enable Rewards
  3. Select Unverified and login with non-KYC'd Gemini account
  4. Confirm no BAT balance shows in panel, on brave://rewards page, etc
  5. Confirm error message is shown indicating that KYC must be completed
example example
Screen Shot 2022-04-26 at 9 13 48 PM Screen Shot 2022-04-26 at 5 17 50 PM

PASS - Gemini Disconnect/Reconnect

Screen Shot 2022-04-25 at 2 57 31 PM

PASS - Switch from Uphold to Gemini (Disconnect/Reconnect)

  1. Clean profile, non-Japan region, enable staging env and relaunch as necessary
  2. Enable Rewards
  3. Select Unverified and login with Uphold KYC'd account
  4. Confirm UI shows Uphold branding and balance
  5. From "Verified" drop down on panel or brave://rewards page, explicitly disconnect from Uphold
  6. Confirm UI returns to pre-connected state ("Unverified" is shown, no balance)
  7. Select Unverified and login with Gemini KYC'd account
  8. Confirm UI shows Gemini branding and balance

PASS - Simulate natural Disconnect

example example example
Screen Shot 2022-04-25 at 2 57 07 PM Screen Shot 2022-04-25 at 2 56 07 PM Screen Shot 2022-04-25 at 2 57 31 PM

PASS - One-Time Tips - KYC'd Gemini (tipping from Gemini KYC wallet to Gemini creator will take time, this is known/expected)

Screen Shot 2022-04-25 at 3 19 29 PM

PASS - Userdrain - Claim Ad grant before linking Gemini

  1. Clean profile, non-Japan region, enable staging env + verbose logs and relaunch as necessary
  2. Enable Rewards
  3. Restore rewards account (backup words) with a waiting Ad grant
  4. Claim the Ad grant
  5. Select Unverified and login with a Gemini account
  6. After successful connection/linking, confirm the Ad grant BAT is sent to Gemini account using logs and brave://rewards-internals
example example example example
Screen Shot 2022-04-26 at 4 52 05 PM Screen Shot 2022-04-26 at 5 02 11 PM Screen Shot 2022-04-26 at 5 03 57 PM Screen Shot 2022-04-26 at 5 04 54 PM

PASS - Confirm Uphold still able to be connected in non-JP region + UI checks

  1. Clean profile, non-Japan region, enable staging env and relaunch as necessary
  2. Enable Rewards
  3. Select Unverified and login with KYC'd Uphold account
  4. Check rewards UI (panel, brave://rewards, etc) and confirm Uphold branding is observed
  5. Confirm when you click on "Verified" text on panel the drop down shows Uphold branding
  6. Confirm when you click on "Verified" text on brave://rewards page the drop down shows Uphold branding
  7. Confirm expected noted BAT balance shows in panel, on brave://rewards page, and in tip banners
  8. Confirm able to do a one-time tip to an Uphold creator and balance decreases as expected
example example example example example example
Screen Shot 2022-04-26 at 3 57 52 PM Screen Shot 2022-04-26 at 3 58 16 PM Screen Shot 2022-04-26 at 3 58 52 PM Screen Shot 2022-04-26 at 4 00 03 PM Screen Shot 2022-04-26 at 4 06 27 PM Screen Shot 2022-04-26 at 4 05 49 PM

PASS - Uphold - minimum BAT required

  1. Clean profile, non-Japan region, enable staging env and relaunch as necessary
  2. Enable Rewards
  3. Click on "Unverified" from panel or brave://rewards
  4. Click thru the verification modal and confirm you see message about 2 BAT required (existing functionality)
  5. Claim UGP grant
  6. Click on "Unverified" again
  7. Confirm verification popup has Uphold and clicking on Uphold option takes you to Uphold sandbox
example example
Screen Shot 2022-04-26 at 4 10 31 PM Screen Shot 2022-04-26 at 4 11 11 PM

PASS - One-Time Tips - KYC'd Uphold

Reference: Tipping Matrix - https://docs.google.com/spreadsheets/d/1WbCD6Hgm0ASThJqlmkVvhwmMGwg5Vtz9X5_t5rR15_0/edit#gid=0

  1. Clean profile, non-Japan region, enable staging env and relaunch as necessary
  2. Enable Rewards
  3. Connect a KYC'd Uphold account
  4. Confirm balance is shown in UI.
  5. Attempt to tip an unverified publisher. Confirm this publisher shows the messaging on panel/tip banners that indicates they are not verified.
  6. Confirm tip goes to "Pending" list, is not deducted from balance, and does not show in "Tips" on brave://rewards
  7. Attempt to tip a connected, but not KYC'd publisher. Confirm this publisher shows messaging on panel/banners indicating they cannot accept tips.
  8. Confirm tip is not deducted from balance, and does not show in "Tips" on brave://rewards
  9. Attempt to tip a KYC'd Uphold publisher. Confirm this publisher does not show any messaging on panel/banners indicating they cannot accept tips.
  10. Confirm tip is deducted from balance and shows in "Tips" on brave://rewards. Confirm the tip shows on Contributions page of brave://rewards-internals.
  11. Confirm both the tip amount (95%) and the Brave fee (5%) are sent by viewing the logs.
  12. Attempt to tip a KYC'd bitFlyer publisher. Confirm this publisher shows messaging on panel/banners indicating they cannot accept tips.
  13. Confirm tip is not deducted from balance, and does not show in "Tips" on brave://rewards
  14. Attempt to tip a KYC'd Gemini publisher. Confirm this publisher shows messaging on panel/banners indicating they cannot accept tips.
  15. Confirm tip is not deducted from balance, and does not show in "Tips" on brave://rewards
Screen Shot 2022-04-26 at 9 19 21 PM

PASS - Confirm bitFlyer still able to be connected in JP region + UI checks

  1. Clean profile, Japan region, enable staging env and relaunch as necessary
  2. Enable Rewards
  3. Select Unverified and login with bitFlyer account from one of the "Basic cases" section of https://docs.google.com/spreadsheets/d/1ko9q8IslO6R7FT37IqNoJG5zBRfPoQbBlu2MnHbSORE/edit#gid=0
  4. Check rewards UI (panel, brave://rewards, etc) and confirm bitFlyer branding is observed
  5. Confirm when you click on "Verified" text on panel the drop down shows bitFlyer branding
  6. Confirm when you click on "Verified" text on brave://rewards page the drop down shows bitFlyer branding
  7. Confirm expected noted BAT balance shows in panel, on brave://rewards page, and in tip banners
  8. Confirm able to do a one-time tip to a bitFlyer creator and balance decreases as expected (note, if you wait more than 7 minutes since logging in, you will be disconnected, this is a feature of staging env to allow testing of simulated disconnect)
example example example
Screen Shot 2022-04-26 at 6 02 14 PM Screen Shot 2022-04-26 at 6 19 02 PM Screen Shot 2022-04-26 at 6 20 32 PM

PASS - bitFlyer Auto Contribute

  1. Clean profile, Japan region, enable staging env and relaunch as necessary
  2. Enable Rewards
  3. Select Unverified and login with bitFlyer KYC'd account
  4. Confirm on brave://rewards that there is no Auto Contribute section
  5. Confirm AC is not listed on panel either
Screen Shot 2022-04-26 at 6 22 37 PM

PASS - One-Time Tips - Anon (no Gemini, Uphold, or bitFlyer)

Reference: Tipping Matrix - https://docs.google.com/spreadsheets/d/1WbCD6Hgm0ASThJqlmkVvhwmMGwg5Vtz9X5_t5rR15_0/edit#gid=0

  1. Clean profile, non-Japan region, enable staging env and relaunch as necessary
  2. Enable Rewards
  3. Do not connect a KYC'd Gemini or Uphold account
  4. Claim UGP grant (anon status)
  5. Attempt to tip an unverified publisher. Confirm this publisher shows the messaging on panel/tip banners that indicates they are not verified.
  6. Confirm tip goes to "Pending" list, is not deducted from balance, and does not show in "Tips" on brave://rewards
  7. Attempt to tip a connected, but not KYC'd publisher. Confirm this publisher does not show any messaging on panel/banners indicating they cannot accept tips.
  8. Confirm tip is deducted from balance and shows in "Tips" on brave://rewards. Confirm the tip shows on Contributions page of brave://rewards-internals.
  9. Attempt to tip a KYC'd Uphold publisher. Confirm this publisher does not show any messaging on panel/banners indicating they cannot accept tips.
  10. Confirm tip is deducted from balance and shows in "Tips" on brave://rewards. Confirm the tip shows on Contributions page of brave://rewards-internals.
  11. Attempt to tip a KYC'd bitFlyer publisher. Confirm this publisher does not show any messaging on panel/banners indicating they cannot accept tips.
  12. Confirm tip is deducted from balance and shows in "Tips" on brave://rewards. Confirm the tip shows on Contributions page of brave://rewards-internals.
  13. Attempt to tip a KYC'd Gemini publisher. Confirm this publisher does not show any messaging on panel/banners indicating they cannot accept tips.
  14. Confirm tip is deducted from balance and shows in "Tips" on brave://rewards. Confirm the tip shows on Contributions page of brave://rewards-internals.
Screen Shot 2022-04-26 at 6 32 02 PM

@Uni-verse Uni-verse added QA/In-Progress Indicates that QA is currently in progress for that particular issue QA Pass - Android Tab and removed QA/In-Progress Indicates that QA is currently in progress for that particular issue labels Apr 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

14 participants