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 directly to Twitch channels. #1814

Closed
jnki opened this issue Oct 23, 2018 · 3 comments · Fixed by brave/brave-core#888
Closed

Cannot tip directly to Twitch channels. #1814

jnki opened this issue Oct 23, 2018 · 3 comments · Fixed by brave/brave-core#888

Comments

@jnki
Copy link

jnki commented Oct 23, 2018

Description

When you visit a verified twitch channel and attempt to "send tip", it only lets you send a tip to twitch.tv, not the specific channel.

Steps to Reproduce

  1. Visit a brave verified twitch channel (mine as example)
  2. Click on Brave Rewards button

Actual result:

image

Expected result:

Expect to see correct url for the specific twitch channel with verified badge, and be able to send tip to it.

Reproduces how often:

Easily reproduced

Brave version (brave://version info)

Brave | 0.55.20 Chromium: 70.0.3538.67 (Official Build) (64-bit)
Revision | 9ab0cfab84ded083718d3a4ff830726efd38869f-refs/branch-heads/3538@{#1002}
OS | Windows 10 64bit 1809

@NejcZdovc
Copy link
Contributor

thank you for your report. Will close this one as we will add support for twitch in with this one #1282

@NejcZdovc NejcZdovc added closed/duplicate Issue has already been reported feature/rewards labels Oct 24, 2018
@NejcZdovc NejcZdovc added this to the Dupe / Invalid / Not actionable milestone Oct 24, 2018
@mandar-brave
Copy link

@NejcZdovc reopening as #1282 did not solve for Twitch

@mandar-brave mandar-brave reopened this Oct 31, 2018
@mandar-brave mandar-brave removed this from the Dupe / Invalid / Not actionable milestone Oct 31, 2018
@mandar-brave mandar-brave added feature/rewards and removed closed/duplicate Issue has already been reported labels Oct 31, 2018
@NejcZdovc NejcZdovc added the priority/P3 The next thing for us to work on. It'll ride the trains. label Oct 31, 2018
@NejcZdovc NejcZdovc added priority/P2 A bad problem. We might uplift this to the next planned release. and removed priority/P3 The next thing for us to work on. It'll ride the trains. labels Oct 31, 2018
@bbondy bbondy added this to the 1.x Backlog milestone Nov 5, 2018
@NejcZdovc NejcZdovc assigned jasonrsadler and unassigned NejcZdovc Nov 8, 2018
@LaurenWags
Copy link
Member

LaurenWags commented Feb 12, 2019

Verified passed with

Brave 0.60.26 Chromium: 72.0.3626.96 (Official Build) beta(64-bit)
Revision 84098ee7ef8622a9defc2ef043cd8930b617b10e-refs/branch-heads/3626@{#836}
OS Mac OS X

Verification passed on

Brave 0.60.26 Chromium: 72.0.3626.96 (Official Build) beta(64-bit)
Revision 84098ee7ef8622a9defc2ef043cd8930b617b10e-refs/branch-heads/3626@{#836}
OS Linux

Verified that:

  • unverified publishers show generic twitch icon in Auto-Contribute table, Tips table, and Rewards Panel.
  • verified publishers show their favicon in Auto-Contribute table, Tips table and Rewards Panel
  • unverified publishers show generic icon and banner on tipping banner
  • verified publishers show custom icon and banner on tipping banner

Encountered #3330 (comment)

Verification passed on

Brave 0.60.27 Chromium: 72.0.3626.96 (Official Build) beta (64-bit)
Revision 84098ee7ef8622a9defc2ef043cd8930b617b10e-refs/branch-heads/3626@{#836}
OS Windows 10
  • Verified that unverified publishers show generic twitch icon in Auto-Contribute table, Tips table, and Panel.
    image
    image

  • Verified that unverified publishers do not show favicon on tipping banner
    image

  • Verified that verified publishers show their favicon in Auto-Contribute table, Tips table, and Panel
    image
    image

  • Verified that verified publishers do show a favicon on tipping banner

image

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.

9 participants