-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Grant virtualization - Phase 1 #6078
Comments
Additional Test Plan for grant virtualization:
We should also verify that existing User Wallet functionality is still working as expected:
|
Verified passed using
PASS - Scenario 1: Using a self-funded anon wallet, verify you are able to
PASS - Scenario 2: Using an anon wallet funded only via a virtual grant (such as UGP), verify you are able to
PASS - Scenario 3: Using an anon wallet funded with a mix of virtual and non-virtual BAT (such as self funded + UGP grant), verify you are able to
PAS - Scenario 4: Using an anon wallet funded with a mix of old and new grants (such as UGP grant + old ad grant), verify you are able to
PASS - Scenario 5: Using a fully KYC'd user wallet, verify you are able to
Logged issue for trouble with connected but not KYC'd accounts for tip, recurring tip, AC (Scenario 6): #7999 Waiting for sandbox account to be verified - Scenario 7: Confirm "Claim" process for self-funded BAT is still working
Verification passed on
PASS - Scenario 1: Using a self-funded anon wallet, verify you are able to
PASS - Scenario 2: Using an anon wallet funded only via a virtual grant (such as UGP), verify you are able to
PASS - Scenario 3: Using an anon wallet funded with a mix of virtual and non-virtual BAT (such as self funded + UGP grant), verify you are able to
PAS - Scenario 4: Using an anon wallet funded with a mix of old and new grants (such as UGP grant + old ad grant), verify you are able to
PASS - Scenario 5: Using a fully KYC'd user wallet, verify you are able to
Scenario 6: Encountered issue #7999 for trouble with connected accounts for tip, recurring tip, AC. Scenario 7: Confirm "Claim" process for self-funded BAT is still working
Verification passed on
PASS - Scenario 1: Using a self-funded anon wallet, verify you are able to PASS - Scenario 2: Using an anon wallet funded only via a virtual grant (such as UGP), verify you are able to PASS - Scenario 3: Using an anon wallet funded with a mix of virtual and non-virtual BAT (such as self funded + UGP grant), verify you are able to PASS - Scenario 4: Using an anon wallet funded with a mix of old and new grants (such as UGP grant + old ad grant), verify you are able to PASS - Scenario 5: Using a fully KYC'd user wallet, verify you are able to Scenario 7: Confirm "Claim" process for self-funded BAT is still working
|
Test Plan
Perform test plan from brave/brave-core#3918
Perform additional test plan from #6078 (comment)
In phase 1 we will not yet remove old contribution flow. We will still ping server for balance, but we will only get user funds in brave wallet back. So we will need to combine 3 things now: grants (blinded tokens), user funds (brave wallet) and external wallet (uphold). Sum of this 3 will give you total.
For contribution user will have two options on how to do things. If we will detect that you have blinded tokens we will use that. If you have only portion that you can use we will first use blinded tokens and then take user funds in brave wallet and at the last step external wallet if you have it connected. If you don't have blinded tokens we will first check user funds and then external wallet. User funds and external wallet will go through the old process for now.
The text was updated successfully, but these errors were encountered: