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] Implement Tip / Contribution Banner 3.0 #28149

Closed
Miyayes opened this issue Jan 30, 2023 · 5 comments · Fixed by brave/brave-core#18423
Closed

[Android] Implement Tip / Contribution Banner 3.0 #28149

Miyayes opened this issue Jan 30, 2023 · 5 comments · Fixed by brave/brave-core#18423
Assignees
Labels
feature/P2P-Send feature/rewards OS/Android Fixes related to Android browser functionality priority/P2 A bad problem. We might uplift this to the next planned release. QA Pass - Android ARM QA/Yes release-notes/include

Comments

@Miyayes
Copy link
Contributor

Miyayes commented Jan 30, 2023

Description

Android version of: #28148

Note: This issue excludes the Android Tablet implementation. Android Tablet implementation will be done separately: #30371

@Miyayes Miyayes added feature/rewards OS/Android Fixes related to Android browser functionality feature/P2P-Send labels Jan 30, 2023
@Miyayes Miyayes added this to New Issues in Brave Rewards via automation Jan 30, 2023
@Miyayes Miyayes added the priority/P2 A bad problem. We might uplift this to the next planned release. label Feb 1, 2023
@Miyayes Miyayes moved this from New Issues to Backlog in Brave Rewards Feb 1, 2023
@Miyayes Miyayes added the QA/Yes label Feb 10, 2023
@Miyayes Miyayes moved this from Backlog to In progress in Brave Rewards Feb 27, 2023
Brave Rewards automation moved this from In progress to Done May 13, 2023
@brave-builds brave-builds added this to the 1.53.x - Nightly milestone May 13, 2023
@LaurenWags
Copy link
Member

@brave/qa-team when testing this issue on Android, please also verify updated tweet text from #30331.

@LaurenWags
Copy link
Member

@brave/qa-team added QA/Test-All-Device-Types since this is new UI so we should verify on phone and tablet.

@GeetaSarvadnya GeetaSarvadnya added the QA/In-Progress Indicates that QA is currently in progress for that particular issue label Jun 7, 2023
@GeetaSarvadnya
Copy link

GeetaSarvadnya commented Jun 7, 2023

Verification PASSED on Vivo X70 Pro version 12 running Bravemonoarm64.apk_1.53.83

User State 1: Rewards not enabled - No change in 1.53 works same as RC 1.52

There is no inline tip icon in RC 1.52.x and 1.53.x for the github/reddit/twitter pubs

User State 2: Unconnected - No change in 1.53 works same as RC 1.52

There is no inline tip icon in RC 1.52.x and 1.53.x for the github/reddit/twitter pubs

User State 3: Connected - PASSED
  • When the tip panel is opened,
    • The user's BAT balance for their provider should be displayed.
Dark Light
image image
  • When the tip panel is opened and the user is logged out from their wallet provider,
    • A login message should be displayed,
    • And a "Log In" button should be displayed that will take the user to the wallet provider's authorization URL.
Uphold Dark Uphold Light Gemini Dark Gemini Light bitFlyer Dark bitFlyer Light
Screenshot_20230609_180702 Screenshot_20230609_181036 Screenshot_20230709_110130 Screenshot_20230709_110151 Screenshot_20230709_112402 Screenshot_20230709_112340
  • When the tip panel is opened and the publisher does not have the same wallet provider as the user,
    • A "mismatch" message should be displayed.
Uphold Dark Uphold Light Gemini Dark Gemini Light bitFlyer Dark bitFlyer Light
Screenshot_20230609_181327 Screenshot_20230609_181237 Screenshot_20230709_105918 Screenshot_20230709_105851 Screenshot_20230709_111346 Screenshot_20230709_111238
  • When "Send" is clicked with a valid amount,
    • The tip should be sent and the success UI should be displayed,
    • And the "Share" button should open Twitter with a "new post" prompt.
Success Dark UI Tweet from Dark UI Success Light UI Tweet from Light UI
Screenshot_20230608_190003 Screenshot_20230608_190012 Screenshot_20230608_190039 Screenshot_20230608_190046
  • When "Send" is clicked and the monthly toggle is set,
    • A one-time tip should be sent,
    • And the success UI should be displayed
    • And a monthly contribution should be added to the "Monthly Contributions" list visible on the Rewards page.
Example Example Example Example Example
Screenshot_20230608_201311 Screenshot_20230608_201322 Screenshot_20230608_201349 Screenshot_20230608_201358 Screenshot_20230608_201438
  • When the tip panel is opened and a monthly contribution is already set for the publisher,
    • The tip panel should display a message indicating that a monthly contribution is already set.
Dark UI panel Dark UI banner Light UI panel Light UI banner
Screenshot_20230608_201631 Screenshot_20230608_201729 Screenshot_20230608_201911 Screenshot_20230608_201922
  • Advanced my computer clock to next monthly contribution date (Jun 12 per Monthly Contributions section of brave://rewards
    • Confirmed monthly tip was sent as expected
    • Confirmed next monthly tip was scheduled per Monthly Contributions section of brave://rewards)
Example Example Example
Screenshot_20230708_202011 Screenshot_20230708_202043 Screenshot_20230708_202110
  • When the Rewards panel is opened for an unverified publisher,
    • A message should be displayed stating that the publisher is not verified,
    • And the "Send Contribution" button should be disabled.
Dark Light
Screenshot_20230709_104843 Screenshot_20230709_104828
  • When the Rewards panel is opened for a verified publisher,
    • The "Send Contribution" button should be enabled.
Dark Light
Screenshot_20230708_203131 Screenshot_20230708_203145
  • When an in-line tip icon is clicked for an unverified publisher, - NA
    • The Rewards panel should be opened with a message indicating the publisher is not verified,
    • And the "Send Contribution" button should be disabled.

Note : There is no inline tip icon in RC 1.52.x and 1.53.x for the github/reddit/twitter pubs

  • When an in-line tip icon is clicked for a verified publisher, - NA
    • The tip panel should be opened.

Note : There is no inline tip icon in RC 1.52.x and 1.53.x for the github/reddit/twitter pubs

  • When an error is encountered when creating a transaction with the wallet provider.
    • An error message should be displayed,
    • And a "Try Again" button should be displayed.
Uphold Dark Uphold Light Gemini Dark Gemini Light bitFlyer Dark bitFlyer Light
Screenshot_20230708_211825 Screenshot_20230708_211936 Screenshot_20230709_110721 Screenshot_20230709_110659 Screenshot_20230709_111421 Screenshot_20230709_111444
User State 4: Grandfathered Unconnected - PASSED
  • When the tip panel is opened,
    • The user's vBAT balance should be displayed.
Dark Light
Screenshot_20230709_114553 Screenshot_20230709_114525
Dark - Uphold Light - Uphold Dark - Gemini Light - Gemini Dark - bitFlyer Light - bitFlyer
Screenshot_20230709_114553 Screenshot_20230709_114525 Screenshot_20230709_114808 Screenshot_20230709_114838
  • When "Send" is clicked with a valid amount,
    • The tip should be sent and the success UI should be displayed,
    • And the "Share" button should open Twitter with a "new post" prompt.
Success Dark UI Tweet from Dark UI Success Light UI Tweet from Light UI
Screenshot_20230709_115311 Screenshot_20230709_115353 Screenshot_20230709_115435 Screenshot_20230709_115441
  • When "Send" is clicked and the monthly toggle is set,
    • A one-time tip should be sent,
    • And the success UI should be displayed
    • And a monthly contribution should be added to the "Monthly Contributions" list visible on the Rewards page.
Example Example Example Example Example
Screenshot_20230709_120038 Screenshot_20230709_120043 Screenshot_20230709_120052 Screenshot_20230709_120058 Screenshot_20230709_120119
  • When the tip panel is opened and a monthly contribution is already set for the publisher,
    • The tip panel should display a message indicating that a monthly contribution is already set.
Dark UI panel Dark UI banner Light UI panel Light UI banner
Screenshot_20230709_120132 Screenshot_20230709_120209
  • Advanced my computer clock to next monthly contribution date (Jun 14 per Monthly Contributions section of brave://rewards
    • Confirmed monthly tip was sent as expected
    • Confirmed next monthly tip was scheduled per Monthly Contributions section of brave://rewards)
Example Example Example
Screenshot_20230808_120416 Screenshot_20230808_120420 Screenshot_20230808_120453
  • When the Rewards panel is opened for an unverified publisher,
    • A message should be displayed stating that the publisher is not verified,
    • And the "Send Contribution" button should be disabled.
Dark Light
Screenshot_20230808_120540 Screenshot_20230808_120527
  • When the Rewards panel is opened for a verified publisher,
    • The "Send Contribution" button should be enabled.
Dark Light
Screenshot_20230808_120626 Screenshot_20230808_120646
  • When an in-line tip icon is clicked for an unverified publisher, NA
    • The Rewards panel should be opened with a message indicating the publisher is not verified,
    • And the "Send Contribution" button should be disabled.

there is no inline tip icon in RC 1.52.x and 1.53.x github/reddit/twitter pubs for grandfathered unconnected state

  • When an in-line tip icon is clicked for a verified publisher, NA
    • The tip panel should be opened.

there is no inline tip icon in RC 1.52.x and 1.53.x github/reddit/twitter pubs for grandfathered unconnected state

  • When an error is encountered when creating a transaction,
    • An error message should be displayed,
    • And a "Try Again" button should be displayed.

Note - error message is not displayed and was logged via #30431

Dark Light
Screenshot_20230808_120859 Screenshot_20230808_120835
Additional tests - PASSED
  1. Confirm when a monthly tip is configured for a creator, a one time tip can still be sent:
  • Fresh profile, close and relaunch to pull griffin
  • Join rewards, connect a custodian
  • Navigate to a creator who is verified with your same custodian
  • Set up a monthly tip
  • Confirm tip is sent immediately and one is scheduled for next month
  • Open tip banner for that same creator, see message about monthly contribution
  • Send another tip
  • Confirm tip is sent
Example Example Example Example Example Example
Screenshot_20230609_143556 Screenshot_20230609_143621 Screenshot_20230609_143625 Screenshot_20230609_143656 Screenshot_20230609_143711 Screenshot_20230609_143730
  1. Confirm no more hover text on panel:
  • Fresh profile, close and relaunch to pull griffin
  • Join rewards, connect a custodian
  • Navigate to a creator who is verified with a different custodian than you
  • Example, I have an Uphold wallet and navigated to a Gemini creator
  • Confirm on the panel, there is no hover text when you hover over "Verified creator"
  • Confirm when you select "Send Contribution" button, the contribution/tip banner shows a message about this creator and you are unable to tip them
Example Example
Screenshot_20230609_143817 Screenshot_20230609_143824
  1. Rounded corners - NA
  • Confirm tip banner has rounded corners

In android tip panels are already rounded

Example Example
Screenshot_20230609_144035 Screenshot_20230609_144043
  1. Confirm when clicking in-line tip button on twitter for a verified creator, that the tip banner no longer quotes the tweet

Note: there is no inline tip icon in RC 1.52.x and 1.53.x github/reddit/twitter pubs for grandfathered unconnected state

  1. Pending Contributions - not tested, will be covered in future issues when Pending Contributions code is removed
Figma - PASSED
  1. From the "Contribution Banner/Android" section
  • The following from Figma were generally covered during testing of "User State 3 - Connected" and "User State 4 - Grandfathered Unconnected" with various sites:

    • Custom vs Standard/default images
    • Click away from banner to close it
    • Contribution sent icon/message
    • Light and dark modes
    • Custom amount entry
  • The following from this section of Figma are shown below:

    • Verified creator icon hover text
    • switch between BAT and USD
    • Monthly Contribution tooltip
Creator icon hover (dark) Creator icon hover (light) BAT/USD switch BAT/USD switch Monthly tooltip (dark) Monthly tooltip (light)
Screenshot_20230609_145440 Screenshot_20230609_145412 Screenshot_20230609_145518 Screenshot_20230609_145523 Screenshot_20230609_145602 Screenshot_20230609_145629
  1. From the "Contribution Banner Error/Info States, User can continue" section - all of these were verified under "User State 3 - Connected" and/or "User State 4 - Grandfathered Unconnected" section:
  • User is already sending monthly tips
Dark UI banner Light UI banner
Screenshot_20230609_151944 Screenshot_20230609_151923
  • Creator has different custodian than user, covered the testing in User State 3: Connected case

  • Creator verified - custodian but no web3 wallet (wording of this is incorrect on the figma)
    User State 3: Connected case

Dark UI banner Light UI banner
Screenshot_20230609_152229 Screenshot_20230609_152253
  • User is logged out of their custodian account

covered the testing in User State 3: Connected case

  • Contribution failure

covered the testing in User State 3: Connected case

  1. From the "Contribution Banner Error/Info States, user dead end" section
  • The following from Figma were generally covered during testing of the various user states (1-4):
    - Creator not registered (panel)
Dark Light
Screenshot_20230609_150748 Screenshot_20230609_150801
  - Custodian is not connected to Rewards (panel)
Dark Light
Screenshot_20230609_150601 Screenshot_20230609_150547
  - No custodian available in region (panel)
Dark Light
Screenshot_20230609_150338 Screenshot_20230609_150319
  - User doesn't have enough tokens for transaction
Dark Light
Screenshot_20230609_145946 Screenshot_20230609_145917

@GeetaSarvadnya
Copy link

GeetaSarvadnya commented Jun 9, 2023

@Miyayes There are few differences found while testing the issue on Android ARM. I am assuming these changes are expected, please log an issue if these are not expected. Thanks!

  1. no inline tip icon for both connected and non-connected users in Android I think which is known
  2. No custodian available in region (panel) message is little different in Android, it's not same as Desktop
Desktop Android
image image
  1. Custodian is not connected to Rewards (panel) message is little different in Android, it's not same as Desktop
Desktop Android
image image

cc: @LaurenWags @deeppandya

@LaurenWags
Copy link
Member

fyi @kjozwiak @vlad-hoff @Uni-verse -

Per discussion with @Miyayes @sujitacharya2005 @deeppandya the Tablet implementation of the tip banner has been deferred and will be implemented with #30371.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature/P2P-Send feature/rewards OS/Android Fixes related to Android browser functionality priority/P2 A bad problem. We might uplift this to the next planned release. QA Pass - Android ARM QA/Yes release-notes/include
Projects
Brave Rewards
  
Done
Development

Successfully merging a pull request may close this issue.

7 participants