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

[P3A] Brave.Rewards.TipsState.2 value 4 isn't captured when a recurring tip is set #8301

Closed
srirambv opened this issue Feb 18, 2020 · 1 comment · Fixed by brave/brave-core#8477

Comments

@srirambv
Copy link
Contributor

Description

[P3A] Brave.Rewards.TipsState.2 value 4 isn't captured when a recurring tip is set

Steps to Reproduce

  1. Clean install 1.3.116
  2. Check Brave.Rewards.TipsState.2 value is set to 0
  3. Enable rewards and claim grant
  4. Check Brave.Rewards.TipsSate.2 value is set to 2
  5. Set recurring tip to verified pub
  6. Check Brave.Rewards.TipsState.2 value is set to 2 and not to 4

Actual result:

image

Expected result:

Should update the value without delay

Reproduces how often:

Easy

Brave version (brave://version info)

Brave 1.3.116 Chromium: 80.0.3987.106 (Official Build) (64-bit)
Revision f68069574609230cf9b635cd784cfb1bf81bb53a-refs/branch-heads/3987@{#882}
OS Linux

Version/Channel Information:

  • Can you reproduce this issue with the current release? N/A
  • Can you reproduce this issue with the beta channel? Yes
  • Can you reproduce this issue with the dev channel? Yes
  • Can you reproduce this issue with the nightly channel? Yes

Other Additional Information:

  • Does the issue resolve itself when disabling Brave Shields? NA
  • Does the issue resolve itself when disabling Brave Rewards? NA
  • Is the issue reproducible on the latest version of Chrome? NA

Miscellaneous Information:

cc: @iefremov

@srirambv srirambv added bug priority/P4 Planned work. We expect to get to it "soon". QA/Yes release-notes/exclude features/P3A labels Feb 18, 2020
@rebron rebron added priority/P3 The next thing for us to work on. It'll ride the trains. and removed priority/P4 Planned work. We expect to get to it "soon". labels Feb 18, 2020
@emerick emerick assigned emerick and unassigned iefremov Apr 8, 2021
@emerick emerick added this to the 1.25.x - Nightly milestone Apr 13, 2021
@stephendonner stephendonner added QA/In-Progress Indicates that QA is currently in progress for that particular issue and removed QA/In-Progress Indicates that QA is currently in progress for that particular issue labels Apr 16, 2021
@stephendonner
Copy link

stephendonner commented Apr 16, 2021

Verified PASSED using

Brave 1.25.22 Chromium: 90.0.4430.72 (Official Build) nightly (x86_64)
Revision b6172ef8d07ef486489a4b11b66b2eaeed50d132-refs/branch-heads/4430@{#1233}
OS macOS Version 11.2.3 (Build 20D91)

Steps:

  1. new profile
  2. enabled Rewards, claimed UGP grant
  3. checked and verified Brave.Rewards.TipsState.2 value is set to 2
  4. set recurring tip to verified pub (used https://www.youtube.com/user/JJDSampson/videos)
  5. after recurring monthly tip was process, checked Brave.Rewards.TipsState.2 value confirmed it was set to 4
example example example
Screen Shot 2021-04-16 at 1 08 28 PM Screen Shot 2021-04-16 at 1 17 37 PM Screen Shot 2021-04-16 at 1 12 46 PM

Note: the original steps to reproduce say "Check Brave.Rewards.TipsState.2 value is set to 0" before enabling Rewards, but this is no longer the case, as I think we purposefully don't set anything until we actually enable Rewards, as part of our "don't ping unless the user initiates it" edict :-)


Verification passed on

Brave | 1.25.53 Chromium: 90.0.4430.93 (Official Build) beta (64-bit)
-- | --
Revision | 4df112c29cfe9a2c69b14195c0275faed4e997a7-refs/branch-heads/4430@{#1348}
OS | Windows 10 OS Version 2004 (Build 19041.928)

  • Ensured Brave.Rewards.TipsState.2 value is set to 2 after rewards enable and UGP grants claim
    image
  • Ensured Brave.Rewards.TipsState.2 value is set to 4 after sending a recurring tip to a verified publisher
    [Note: required browser restart the see the value 4 which is expected]
    image

Verification passed on

Brave 1.25.59 Chromium: 90.0.4430.212 (Official Build) beta (64-bit)
Revision e3cd97fc771b893b7fd1879196d1215b622c2bed-refs/branch-heads/4430@{#1429}
OS Ubuntu 18.04 LTS
  • Ensured Brave.Rewards.TipsState.2 value is set to 2 after rewards enable and UGP grants claim
    image
  • Ensured Brave.Rewards.TipsState.2 value is set to 4 after sending a recurring tip to a verified publisher
    image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment