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

Remove "Add funds" buttons in Rewards (remove add_url and withdraw_url from ExternalWallet) #23793

Closed
Miyayes opened this issue Jun 30, 2022 · 6 comments · Fixed by brave/brave-core#16190
Assignees
Labels
feature/rewards OS/Android Fixes related to Android browser functionality OS/Desktop priority/P3 The next thing for us to work on. It'll ride the trains. QA Pass - Android ARM QA Pass-macOS QA/Yes release-notes/exclude rewards-2.0

Comments

@Miyayes
Copy link
Collaborator

Miyayes commented Jun 30, 2022

Description

The "Add funds" button does not serve its purpose very well in Rewards 2.0. We can remove it for now, and be able to clean up the code in quite a few places if it is removed. Can be re-introduced in Rewards 3.0.

@szilardszaloki Has most context.

@Miyayes Miyayes added the priority/P3 The next thing for us to work on. It'll ride the trains. label Jun 30, 2022
@Miyayes Miyayes added the QA/Yes label Aug 3, 2022
@Miyayes Miyayes assigned emerick and unassigned szilardszaloki Nov 16, 2022
@emerick
Copy link
Contributor

emerick commented Nov 29, 2022

@Miyayes Are we dropping the entire "Not enough tokens. Please add funds." section from the tip banner? Or just removing the "Please add funds" portion?

@Miyayes
Copy link
Collaborator Author

Miyayes commented Nov 29, 2022

@emerick Hm, that's a good point. I think it would make sense to just drop "Please add funds" part, and keep the "Not enough tokens" part?

@emerick
Copy link
Contributor

emerick commented Nov 30, 2022

@Miyayes One more question: does this mean removing the "Add funds" notification? For the moment, I've renamed it to an "Insufficient funds" notification, but let me know if you'd like to remove or adjust it.

@emerick
Copy link
Contributor

emerick commented Dec 2, 2022

Just noting here that we decided to remove the "insufficient funds" notifications as they are confusing to users and don't provide much value.

@emerick emerick added this to the 1.48.x - Nightly milestone Dec 7, 2022
@LaurenWags LaurenWags added the QA/In-Progress Indicates that QA is currently in progress for that particular issue label Jan 17, 2023
@LaurenWags
Copy link
Member

LaurenWags commented Jan 17, 2023

Verified with

Brave | 1.48.129 Chromium: 109.0.5414.87 (Official Build) beta (x86_64)
-- | --
Revision | 2dc18eb511c56e012081b4abc9e38c81c885f7d4-refs/branch-heads/5414@{#1241}
OS | macOS Version 12.6.2 (Build 21G320)
Uphold - PASSED

Prior to fix in 1.47.171:

brave://rewards Rewards Panel Tip Banner
1 47 x a 1 47 x b 1 47 x c

After the fix with 1.48.129:
Confirmed "Add funds" removed from wallet panel on brave://rewards, panel from BAT logo, and tip banner (when applicable).

brave://rewards Rewards Panel Tip Banner
1 48 x a 1 48 x b 1 48 x c
Gemini - PASSED

Prior to fix in 1.47.171:

brave://rewards Rewards Panel
Screen Shot 2023-01-17 at 4 26 41 PM Screen Shot 2023-01-17 at 4 26 48 PM

After the fix with 1.48.129:
Confirmed "Add funds" removed from wallet panel on brave://rewards and panel from BAT logo. Unable to check tip banner due to balance being too high (max tip amount is 100 BAT, unable to get Gemini balance under 100 BAT)

brave://rewards Rewards Panel
Screen Shot 2023-01-17 at 4 26 53 PM Screen Shot 2023-01-17 at 4 27 00 PM
bitFlyer - PASSED

Prior to fix in 1.47.171:

brave://rewards Rewards Panel Tip Banner
Screen Shot 2023-01-17 at 4 38 15 PM Screen Shot 2023-01-17 at 4 38 25 PM Screen Shot 2023-01-17 at 4 38 51 PM

After the fix with 1.48.129:
Confirmed "Add funds" removed from wallet panel on brave://rewards, panel from BAT logo, and tip banner (when applicable).

brave://rewards Rewards Panel Tip Banner
Screen Shot 2023-01-17 at 4 38 55 PM Screen Shot 2023-01-17 at 4 39 02 PM Screen Shot 2023-01-17 at 4 39 19 PM

Note - Unable to confirm the "Insufficient Funds" notification was removed (test plan from #3286 did not display the notification on 1.47.x, other tests proved unsuccessful as well). After conferring with Emerick, decided that since notification was removed it was ok to skip confirming this.

@LaurenWags LaurenWags added QA Pass-macOS and removed QA/In-Progress Indicates that QA is currently in progress for that particular issue labels Jan 17, 2023
@stephendonner
Copy link

stephendonner commented Jan 31, 2023

Verified PASSED using Brave 1.48.149, Chromium 109.0.5414.119 on a Google Pixel XL (arm64) running Android 9

Uphold - PASSED

Confirmed + Add Funds was removed from:

  • brave://rewards
  • Rewards panel
  • setting a custom-tip value more than the available BAT automatically reduces it to 100 (in my case), without the same error message as desktop
brave://rewards Rewards panel tip
Screenshot_20230130-212405 Screenshot_20230130-212548 Screenshot_20230131-101250

Gemini - PASSED

Confirmed + Add Funds was removed from:

  • brave://rewards
  • Rewards panel
  • setting a custom-tip value more than the available BAT automatically reduces it to 100 (in my case), without the same error message as desktop
brave://rewards Rewards panel tip
Screenshot_20230131-105903 Screenshot_20230131-105917 Screenshot_20230131-110014

bitFlyer - PASSED

Confirmed + Add Funds was removed from:

  • brave://rewards
  • Rewards panel
  • setting a custom-tip value more than the available BAT automatically reduces it to 100 (in my case), without the same error message as desktop
brave://rewards Rewards panel tip
Screenshot_20230201-043208 Screenshot_20230201-043216 Screenshot_20230201-043301

@stephendonner stephendonner added the QA/In-Progress Indicates that QA is currently in progress for that particular issue label Jan 31, 2023
@stephendonner stephendonner added QA Pass - Android ARM and removed QA/In-Progress Indicates that QA is currently in progress for that particular issue labels Jan 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature/rewards OS/Android Fixes related to Android browser functionality OS/Desktop priority/P3 The next thing for us to work on. It'll ride the trains. QA Pass - Android ARM QA Pass-macOS QA/Yes release-notes/exclude rewards-2.0
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants