-
Notifications
You must be signed in to change notification settings - Fork 2.6k
Wrong twitch channel linked in Auto-Contribute #4045
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
Comments
This also affects |
May also be worth mentioning that this seems to affect VOD only. I am unable to reproduce with live streams. |
Changed static ref function to not param non-const string
Changed static ref function to not param non-const string Address PR comments Moved to single return Use GetPublisherKey to set id
Verification passed on
Used testplan from brave/brave-core#2224. Any channel with underscore will exhibit this issue. I used https://www.twitch.tv/tal_raja/videos. Tested stream and VODs. Encountered #4921 Verification passed on
Verified the test plan from brave/brave-core#2224. Any channel with an underscore will exhibit this issue. Verified https://www.twitch.tv/torch_light. Tested stream and VODs. Activity_Info: Publisher_Info: media_publisher_info: Pending_contributions: Verified pub with an underscore in the middle of user id: contribution_info Verified passed with
|
Description
Wrong twitch channel linked in the Auto-Contribute section of brave://rewards.
Steps to Reproduce
Actual result:
Links to the twitch channel with the initial part of the name (ie. the part before the underscore). For our example channel, it links to https://www.twitch.tv/anatomyz/videos
Expected result:
Links to the the twitch channel you watched. Example: Links to https://www.twitch.tv/anatomyz_2/videos
Reproduces how often:
For a fixed channel, every time you click the link (10/10).
For different channels, reproducing it is more annoying, as you need to either find or create such channels, and channel search of twitch does not seem support underscore. So far, I only found one other channel (leagueofladies_) which has a fitting name and encountered the bug with that channel as well. If there are letters after the underscore, everything works fine
Brave version (brave://version info)
Reproducible on current release:
Not tested
Additional Information
The correct channel name does appear in the list, only the link is wrong. I have no idea if the contribution itself goes to the correct channel or not though.
The text was updated successfully, but these errors were encountered: