-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Wrong publisher in Rewards Panel on youtube #12281
Wrong publisher in Rewards Panel on youtube #12281
Comments
I think this is happening when you go from a YouTube channel page to a custom YouTube URL. We saw something similar with YouTube user URLs during development. YouTube was updating the page on navigation, but certain DOM elements that we relied on for determining where we were (like scripts) remained the same and so were pointing to old data post-navigation. The fix is to always fetch custom URLs, just like we do user URLs. |
Fix will be available in Greaselion |
@LaurenWags and myself ran through the above on staging as per brave/brave-site-specific-scripts#23 (comment) & brave/brave-site-specific-scripts#23 (comment). Pushing these changes live.
As per @emerick, new component should be |
Verification passed on
Verified test plan from brave/brave-site-specific-scripts#23 Verified proper publisher is shown for https://www.youtube.com/channel/UCCs7AQEDwrHEc86r0NNXE_A Verified that after browsing youtube, proper publisher is shown for custom youtube url Verification passed on
Verified test plan from brave/brave-site-specific-scripts#23 Verified proper publisher is shown for https://www.youtube.com/channel/UCCs7AQEDwrHEc86r0NNXE_A Verified that after browsing youtube, proper publisher is shown for custom youtube url Verified passed with
Confirmed expected component version from #12281 (comment) Verified test plan from brave/brave-site-specific-scripts#23 Confirmed expected information in the panel initially: Confirmed expected information in the panel after navigating to a custom URL: Confirmed expected information in the panel after browsing around a bit and using back/forward navigation arrows: |
Sometimes previous publisher shows up in Rewards Panel
Note: This is a regression introduced in 1.16.x
Steps to Reproduce
https://www.youtube.com/channel/UCCs7AQEDwrHEc86r0NNXE_A
Note: Best to check the recording
Note: Proper publisher is shown after page reload
Actual result:
Wrong publisher in Rewards Panel
Expected result:
Proper publisher in Rewards Panel
Reproduces how often:
100% repro rate
Brave version (brave://version info)
Reproducible on
Not reproducible on
cc @brave/legacy_qa @rebron @NejcZdovc
The text was updated successfully, but these errors were encountered: