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

Cannot tip to Twitch channels #4680

Closed
Ravendwyr opened this issue Jun 2, 2019 · 5 comments · Fixed by brave/brave-core#2618
Closed

Cannot tip to Twitch channels #4680

Ravendwyr opened this issue Jun 2, 2019 · 5 comments · Fixed by brave/brave-core#2618

Comments

@Ravendwyr
Copy link

Ravendwyr commented Jun 2, 2019

Description

It looks like #1814 has appeared again as attempting to send a tip via Twitch shows "Twitch.tv" as the tip recipient instead of the channel. YouTube and Website tipping appears to be working as expected however.

Steps to Reproduce

  1. Visit any Twitch channel.
  2. Click on Brave Rewards button.

Actual result:

image

Expected result:

Rewards popup to show if the streamer is verified or not with a button to send a tip.

Reproduces how often:

Easily reproduced on Windows, macOS, and Android.

Brave version (brave://version info)

Version/Channel Information:

  • Can you reproduce this issue with the current release? Yes
  • Can you reproduce this issue with the beta channel? Haven't tested yet
  • Can you reproduce this issue with the dev channel? Haven't tested yet
  • Can you reproduce this issue with the nightly channel? Haven't tested yet

Other Additional Information:

  • Does the issue resolve itself when disabling Brave Shields? No
  • Does the issue resolve itself when disabling Brave Rewards? N/A
  • Is the issue reproducible on the latest version of Chrome? N/A

Miscellaneous Information:

@LaurenWags
Copy link
Member

@EthanCentaurai Are you trying to tip while watching a VOD? if so, you're probably encountering this issue: #3590 (for macOS, Windows, Linux).

Note, Twitch tipping is not implemented for Android yet, that's logged here: https://github.com/brave/browser-android-tabs/issues/1301

@Ravendwyr
Copy link
Author

Hello there. I am watching a live streamed video; not a VOD or a hosted channel or rerun.

@LaurenWags
Copy link
Member

LaurenWags commented Jun 3, 2019

Thanks @EthanCentaurai 😄 Looks like I can reproduce with the following versions:

Brave 0.64.77 Chromium: 74.0.3729.169 (Official Build) (64-bit)
Revision 78e4f8db3ce38f6c26cf56eed7ae9b331fc67ada-refs/branch-heads/3729@{#1013}
OS Mac OS X
Brave 0.65.116 Chromium: 75.0.3770.38 (Official Build) (64-bit)
Revision 3860105745f2b12537da9e9f048f14c3f52ba970-refs/branch-heads/3770@{#618}
OS Mac OS X
Brave 0.66.79 Chromium: 75.0.3770.38 (Official Build) beta(64-bit)
Revision 3860105745f2b12537da9e9f048f14c3f52ba970-refs/branch-heads/3770@{#618}
OS Mac OS X
Brave 0.67.72 Chromium: 75.0.3770.38 (Official Build) dev(64-bit)
Revision 3860105745f2b12537da9e9f048f14c3f52ba970-refs/branch-heads/3770@{#618}
OS Mac OS X

cc @NejcZdovc

Checked an older released version and it stopped working there as well:

Brave 0.64.76 Chromium: 74.0.3729.157 (Official Build) (64-bit)
Revision 7b16107ab85c5364cdcd0b2dea2539a1f2dc327a-refs/branch-heads/3729@{#998}
OS Mac OS X

@btlechowski
Copy link

Reproduced on Windows 10 x64

Brave 0.64.77 Chromium: 74.0.3729.169 (Official Build) (64-bit)
Revision 78e4f8db3ce38f6c26cf56eed7ae9b331fc67ada-refs/branch-heads/3729@{#1013}
OS Windows 10 OS Build 17134.765
Brave 0.65.116 Chromium: 75.0.3770.38 (Official Build) (64-bit)
Revision 3860105745f2b12537da9e9f048f14c3f52ba970-refs/branch-heads/3770@{#618}
OS Windows 10 OS Version 1803 (Build 17134.765)
Brave 0.66.79 Chromium: 75.0.3770.38 (Official Build) beta (64-bit)
Revision 3860105745f2b12537da9e9f048f14c3f52ba970-refs/branch-heads/3770@{#618}
OS Windows 10 OS Version 1803 (Build 17134.765)
Brave 0.67.72 Chromium: 75.0.3770.38 (Official Build) dev (64-bit)
Revision 3860105745f2b12537da9e9f048f14c3f52ba970-refs/branch-heads/3770@{#618}
OS Windows 10 OS Version 1803 (Build 17134.765)

@mandar-brave mandar-brave added the priority/P1 A very extremely bad problem. We might push a hotfix for it. label Jun 4, 2019
@NejcZdovc NejcZdovc self-assigned this Jun 6, 2019
@NejcZdovc NejcZdovc added this to the 0.68.x - Nightly milestone Jun 6, 2019
NejcZdovc added a commit to brave/brave-core that referenced this issue Jun 6, 2019
NejcZdovc added a commit to brave/brave-core that referenced this issue Jun 12, 2019
@NejcZdovc NejcZdovc removed this from the 0.67.x - Dev milestone Jun 13, 2019
@NejcZdovc NejcZdovc added this to the 0.66.x - Beta milestone Jun 13, 2019
NejcZdovc added a commit to brave/brave-core that referenced this issue Jun 14, 2019
@btlechowski
Copy link

btlechowski commented Jun 14, 2019

Verification passed on

Brave 0.65.120 Chromium: 75.0.3770.90 (Official Build) (64-bit)
Revision a6dcaf7e3ec6f70a194cc25e8149475c6590e025-refs/branch-heads/3770@{#1003}
OS Windows 7 Service Pack 1 (Build 7601.24468)

Verified test plan from brave/brave-core#2618.

image

Verified passed with

Brave 0.65.120 Chromium: 75.0.3770.90 (Official Build) (64-bit)
Revision a6dcaf7e3ec6f70a194cc25e8149475c6590e025-refs/branch-heads/3770@{#1003}
OS Mac OS X

Verification passed on

Brave 0.65.120 Chromium: 75.0.3770.90 (Official Build) (64-bit)
Revision a6dcaf7e3ec6f70a194cc25e8149475c6590e025-refs/branch-heads/3770@{#1003}
OS Ubuntu 18.04 LTS

Verified test plan from brave/brave-core#2618.

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

Successfully merging a pull request may close this issue.

6 participants