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

fix(youtube): channel name and viewed content #8362

Merged
merged 14 commits into from May 11, 2024
Merged

Conversation

DMCroww
Copy link
Contributor

@DMCroww DMCroww commented May 8, 2024

Description

Came across instance where channel name and selected tabs were not showing up in Presence (undefined and null respectively)

Line 330: Adjusted an if statement that was causing channel name to be reported as null when an element on page was not found (String.includes('') is always true)

Line 349: Changed querystring to properly find selected tab of viewed content (Featured, Videos, ...) as it was returning undefined

Acknowledgements

Screenshots

Proof showing the creation/modification is working as expected

Screenshot 2024-05-08 19-58-46

Screenshot 2024-05-08 19-59-39

websites/Y/YouTube/presence.ts Outdated Show resolved Hide resolved
Co-authored-by: Dark_Ville <42322979+DarkVillager@users.noreply.github.com>
Signed-off-by: Daniel Lau <32113157+theusaf@users.noreply.github.com>
@theusaf theusaf enabled auto-merge (squash) May 11, 2024 04:56
@theusaf theusaf requested a review from Bas950 May 11, 2024 04:57
@theusaf theusaf merged commit 5eb8c71 into PreMiD:main May 11, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

None yet

4 participants