Adjust unverified publisher note display logic #10242

zenparsing · 2021-09-24T15:18:57Z

Previously, an "unverified tipping note" would appear in the following case:

The user does not have an external wallet
The publisher is registered but does not have a verified wallet provider address.

Per clarification in brave/brave-browser#18137, the user should not see the message in this case on either the rewards panel or the tipping banner.

Submitter Checklist:

I confirm that no security/privacy review is needed, or that I have requested one
There is a ticket for my issue
Used Github auto-closing keywords in the PR description above
Wrote a good PR/commit description
Added appropriate labels (QA/Yes or QA/No; release-notes/include or release-notes/exclude; OS/...) to the associated issue
Checked the PR locally: npm run test -- brave_browser_tests, npm run test -- brave_unit_tests, npm run lint, npm run gn_check, npm run tslint
Ran git rebase master (if needed)

Reviewer Checklist:

A security review is not needed, or a link to one is included in the PR description
New files have MPL-2.0 license header
Adequate test coverage exists to prevent regressions
Major classes, functions and non-trivial code blocks are well-commented
Changes in component dependencies are properly reflected in gn
Code follows the style guide
Test plan is specified in PR before merging

After-merge Checklist:

Test Plan:

Message 1: "This creator is not signed up yet. Any tip you make will remain pending in your wallet and retry automatically for 90 days. Learn more."

Message 2: "This creator is currently not configured to receive tips from your wallet. Any tip you make will remain pending in your wallet and retry automatically for 90 days. Learn more."

Case 1: Not registered

Example staging creator: apple.com

Given that the user has enabled rewards
When the user navigates to a publisher page
And the publisher is not a registered creator
Then the Rewards panel publisher status indicator should display message 1 on hover.

Case 2: Registered but not verified, user not verified

Example staging creator: www.reddit.com/user/jsadler-brave

Given that the user has enabled rewards
And the user does not have a connected external wallet
When the user navigates to a publisher page
And the publisher is a registered creator
And the publisher is not verified with any wallet provider
Then the Rewards panel publisher status indicator should not display a message on hover.

Case 3: Registered but not verified, user verified

Example staging creator: www.reddit.com/user/jsadler-brave

Given that the user has enabled rewards
And the user has a connected external wallet
When the user navigates to a publisher page
And the publisher is a registered creator
And the publisher is not verified with any wallet provider
Then the Rewards panel publisher status indicator should display message 2 on hover.

Case 4: Verified and user is not connected

Example staging creator: laurenwags.github.io

Given that the user has enabled rewards
And has not connected to an external wallet provider
When the user navigates to a publisher page
And the publisher is verified with any wallet provider
Then the Rewards panel publisher status indicator should not display a message on hover

Case 5: Verified with non-matching wallet provider

Given that the user has enabled rewards
And has connected to an external wallet provider
When the user navigates to a publisher page
And the publisher is verified with a different wallet provider
Then the Rewards panel publisher status indicator should display message 2 on hover.

Case 6: Verified with matching wallet provider

Given that the user has enabled rewards
And has connected to an external wallet provider
When the user navigates to a publisher page
And the publisher is verified with the same wallet provider
Then the Rewards panel publisher status indicator should not display a message on hover.

emerick

LGTM

LaurenWags · 2021-09-29T19:52:57Z

Verified using

Brave	1.32.31 Chromium: 94.0.4606.61 (Official Build) nightly (x86_64)
Revision	418b78f5838ed0b1c69bb4e51ea0252171854915-refs/branch-heads/4606@{#1204}
OS	macOS Version 10.15.7 (Build 19H1417)

Verified test plan from #10242 (comment)

Scenario 1 - Not registered - PASSED

Confirmed message 1 is displayed on the tipping banner

Example	Example

Scenario 2 - Registered but not verified, user not verified - PASSED

Confirmed no message is displayed about publisher status

Example	Example

Scenario 3 - Registered but not verified, user verified - PASSED

Confirmed message 2 is displayed about publisher status

Example	Example

Scenario 4 - Verified and user is not connected - PASSED

Confirmed no message is displayed about publisher status

Example	Example

Brave | 1.32.33 Chromium: 94.0.4606.61 (Official Build) nightly (x86_64)
-- | --
Revision | 418b78f5838ed0b1c69bb4e51ea0252171854915-refs/branch-heads/4606@{#1204}
OS | macOS Version 10.15.7 (Build 19H1417)

Scenario 5 - Verified with non-matching wallet provider - PASSED

Confirmed message 2 is displayed about publisher status

Example	Example

Scenario 6 - Verified with matching wallet provider - PASSED

Confirmed no message is displayed about publisher status

Example	Example

Additional checks performed:

Anon wallet (funded via UGP grant) - PASSED

Confirmed tip to unverified creator went to pending list and appropriate message (1) was displayed:

Example	Example	Example	Example

Confirmed able to tip connected, Uphold, Gemini, and bitFlyer KYC'd creators and no messaging was displayed.
Confirmed balance decreased as expected for tip to each of these creators.

Connected Example	Connected Example	Uphold Example	Uphold Example	Gemini Example	Gemini Example	bitFlyer Example	bitFlyer Example	Tips List	Wallet Panel

Uphold wallet (funded via Uphold BAT on staging env) - PASSED

Confirmed tip to unverified creator went to pending list and appropriate message (1) was displayed:

Example	Example	Example	Example

Confirmed tip to connected, Gemini, and bitFlyer creators went to pending list and appropriate message (2) was displayed:

Connected Example	Connected Example	Gemini Example	Gemini Example	bitFlyer Example	bitFlyer Example	Wallet Panel	Pending Tips List

Confirmed able to tip Uphold KYC'd creator and no messaging was displayed.
Confirmed balance decreased as expected for tip.

Example	Example	Example	Example

bitFlyer wallet (funded via bitFlyer BAT on staging env) - PASSED

Confirmed tip to unverified creator went to pending list and appropriate message (1) was displayed:

Example	Example	Example	Example

Confirmed tip to connected, Gemini, and Uphold creators went to pending list and appropriate message (2) was displayed:

Connected Example	Connected Example	Gemini Example	Gemini Example	Uphold Example	Uphold Example	Wallet Panel	Pending Tips List

Confirmed able to tip bitFlyer KYC'd creator and no messaging was displayed.
Confirmed balance decreased as expected for tip.

Example	Example	Example	Example

Gemini wallet (funded via Gemini BAT on staging env) - PASSED

Confirmed tip to unverified creator went to pending list and appropriate message (1) was displayed:

Example	Example	Example	Example

Confirmed tip to connected, bitFlyer, and Uphold creators went to pending list and appropriate message (2) was displayed:

Connected Example	Connected Example	bitFlyer Example	bitFlyer Example	Uphold Example	Uphold Example	Wallet Panel	Pending Tips List

Confirmed able to tip Gemini KYC'd creator and no messaging was displayed.
Confirmed balance decreased as expected for tip.

Example	Example	Example	Example

Adjust unverified publisher note display logic

61e1e30

zenparsing requested a review from a team as a code owner September 24, 2021 15:18

zenparsing requested review from jumde and emerick September 24, 2021 15:18

github-actions bot assigned zenparsing Sep 24, 2021

emerick approved these changes Sep 24, 2021

View reviewed changes

zenparsing merged commit 54a5735 into master Sep 24, 2021

zenparsing deleted the ksmith-unverified-note-logic branch September 24, 2021 21:05

zenparsing added this to the 1.32.x - Nightly milestone Sep 29, 2021

LaurenWags mentioned this pull request Sep 29, 2021

Show correct notice on tipping banner for unregistered creators #10137

Merged

24 tasks

zenparsing added a commit that referenced this pull request Sep 29, 2021

Uplift of #10242 (squashed) to beta

f7c34c8

zenparsing mentioned this pull request Sep 29, 2021

Show correct notice on tipping banner for unregistered creators (uplift to 1.31.x) #10271

Merged

7 tasks

LaurenWags mentioned this pull request Oct 4, 2021

Do not show tipping notice for unverified rewards user case brave/brave-browser#18341

Closed

GeetaSarvadnya mentioned this pull request Oct 4, 2021

Add publisher status hover note to Rewards panel brave/brave-browser#17941

Closed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Adjust unverified publisher note display logic #10242

Adjust unverified publisher note display logic #10242

zenparsing commented Sep 24, 2021 •

edited

Loading

emerick left a comment

LaurenWags commented Sep 29, 2021 •

edited

Loading

Adjust unverified publisher note display logic #10242

Adjust unverified publisher note display logic #10242

Conversation

zenparsing commented Sep 24, 2021 • edited Loading

Submitter Checklist:

Reviewer Checklist:

After-merge Checklist:

Test Plan:

Case 1: Not registered

Case 2: Registered but not verified, user not verified

Case 3: Registered but not verified, user verified

Case 4: Verified and user is not connected

Case 5: Verified with non-matching wallet provider

Case 6: Verified with matching wallet provider

emerick left a comment

Choose a reason for hiding this comment

LaurenWags commented Sep 29, 2021 • edited Loading

zenparsing commented Sep 24, 2021 •

edited

Loading

LaurenWags commented Sep 29, 2021 •

edited

Loading