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

sometimes wrong twitch publisher info is shown including verified logo/text #3423

Closed
LaurenWags opened this issue Feb 19, 2019 · 10 comments
Closed
Labels
closed/stale Issue is no longer relevant, perhaps because the feature it refers to has been deprecated.

Comments

@LaurenWags
Copy link
Member

Description

Sometimes Brave Rewards Panel will show incorrect twitch publisher info, this is particularly bad because the panel will sometimes show the wrong publisher as verified, which is misleading.

Steps to Reproduce

  1. Clean profile
  2. Launch Brave, enable Rewards
  3. Open a tab and navigate to twitch.tv/directory
  4. Click on a subject like Fortnite
  5. Click on a name (not stream) of one of the people in this category (this will take you to their videos page, not their current live stream)
  6. Click on a Video to start playing a VOD for this publisher who we’re going to call pub1.
  7. Click on the panel (chances are they’re not a verified publisher).
  8. Click on search box, type in neoness007, click on this person under the 'Channels' heading in the drop down search results box.
  9. Once stream starts (wait until after pre-roll ad if one shows), click on BAT logo.

Actual result:

  1. Info for previous publisher (pub1) is shown and incorrect publisher is shown as verified with custom favicon but in fact, you're on a different publisher page. Note - the person who's page you are on is actually verified, not pub1.
    screen shot 2019-02-19 at 5 55 44 pm
    For reference, this is what I see when I access the Brave Rewards panel while on pub1's channel:
    screen shot 2019-02-19 at 6 02 06 pm

Additionally, if you do try to tip while seeing the wrong info in panel, you incorrectly see pub1 info on the tip banner. If you tip them, they’re listed in the tips table on brave://rewards
screen shot 2019-02-19 at 5 56 28 pm

Now if you go to brave://rewards, a-c table shows pub1 listed twice - once as verified and once as not verified.
screen shot 2019-02-19 at 6 04 17 pm

Expected result:

Incorrect info shouldn't be shown anywhere - panel, a-c table, tip banner, tips table.

Reproduces how often:

consistently with steps above

Brave version (brave://version info)

Brave 0.60.44 Chromium: 72.0.3626.109 (Official Build) (64-bit)
Revision fae8db7ab9280fa6704a59980263c804f809ebd5-refs/branch-heads/3626@{#857}
OS Mac OS X

Reproducible on current release: n/a - 0.59.35 doesn't have twitch tipping

  • Does it reproduce on brave-browser dev/beta builds? unsure

Website problems only:

  • Does the issue resolve itself when disabling Brave Shields? n/a
  • Is the issue reproducible on the latest version of Chrome? n/a

Additional Information

cc @brave/legacy_qa to verify on other platforms
cc @mandar-brave @rebron @NejcZdovc

@btlechowski
Copy link

Nice find @LaurenWags !

Reproduced on Windows 7

Brave 0.60.44 Chromium: 72.0.3626.109 (Official Build) (64-bit)
Revision fae8db7ab9280fa6704a59980263c804f809ebd5-refs/branch-heads/3626@{#857}
OS Windows 7

image

@mandar-brave mandar-brave added the priority/P1 A very extremely bad problem. We might push a hotfix for it. label Feb 20, 2019
@mandar-brave
Copy link

Moving priority to P1 considering it will cause a lot of negative community feedback

@NejcZdovc NejcZdovc self-assigned this Feb 20, 2019
@NejcZdovc NejcZdovc added priority/P2 A bad problem. We might uplift this to the next planned release. and removed priority/P1 A very extremely bad problem. We might push a hotfix for it. labels Feb 20, 2019
@rebron
Copy link
Collaborator

rebron commented Feb 27, 2019

@NejcZdovc did this get fix with the last refactor that landed (or is landing) in 61.x?

@kjozwiak
Copy link
Member

kjozwiak commented Feb 27, 2019

Labelled this as release/blocking as we should really get this into 0.61.x. @jsecretan would love to get your thoughts on this one and how severe you think the issue is. If @NejcZdovc mentions that the publisher refactor that's in 0.61.x fixes this, then it's not a problem. However if the refactor doesn't handle this issue, we'll need to come up with a timeline.

This might end up being bad in terms of PR when some folks notice their tips went to the wrong publisher.

Edited: Removed release/blocking until we get more info from @NejcZdovc & @jsecretan.

@NejcZdovc NejcZdovc removed their assignment Feb 27, 2019
@NejcZdovc NejcZdovc added priority/P1 A very extremely bad problem. We might push a hotfix for it. and removed priority/P2 A bad problem. We might uplift this to the next planned release. labels Feb 27, 2019
@NejcZdovc
Copy link
Contributor

I just tried on the v0.61.37 and I can still reproduce it. I tried it on the latest master and I can't reproduce it anymore.

@mandar-brave
Copy link

@NejcZdovc do we need to uplift a fix to 0.61x; that is the ask. If we can we should.
cc @kjozwiak

@NejcZdovc
Copy link
Contributor

@mandar-brave not sure what fixed it 😃

@NejcZdovc NejcZdovc added priority/P2 A bad problem. We might uplift this to the next planned release. and removed priority/P1 A very extremely bad problem. We might push a hotfix for it. labels Mar 6, 2019
@NejcZdovc
Copy link
Contributor

@brave/legacy_qa can you please test this one on the latest nightly (0.63) or dev when we migrate today? Thank you

@NejcZdovc NejcZdovc added priority/P3 The next thing for us to work on. It'll ride the trains. and removed priority/P2 A bad problem. We might uplift this to the next planned release. labels Mar 7, 2019
@LaurenWags
Copy link
Member Author

@NejcZdovc tried on below build and issue still reproduces.

Brave 0.63.4 Chromium: 73.0.3683.67 (Official Build) dev (64-bit)
Revision a83fd4f3207ae83412d329a9ca1239dd1e068345-refs/branch-heads/3683@{#760}
OS Mac OS X

@Miyayes
Copy link
Collaborator

Miyayes commented Oct 8, 2021

I think this should be resolved with new panel. Also, there's likely a newer version of this issue as well.

@Miyayes Miyayes closed this as completed Oct 8, 2021
@kjozwiak kjozwiak added the closed/stale Issue is no longer relevant, perhaps because the feature it refers to has been deprecated. label Oct 26, 2021
@kjozwiak kjozwiak removed bug feature/rewards priority/P3 The next thing for us to work on. It'll ride the trains. QA/Yes QA/Test-Plan-Specified labels Oct 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
closed/stale Issue is no longer relevant, perhaps because the feature it refers to has been deprecated.
Projects
None yet
Development

No branches or pull requests

8 participants