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(downloader): use the upgrade button to check for premium status #2987

Merged
merged 1 commit into from
Feb 17, 2025

Conversation

franz-dc
Copy link
Contributor

@franz-dc franz-dc commented Feb 16, 2025

Fixes #2985 and also fixes another 304 error when downloading (#2888).

This uses the "Upgrade" button to check for YouTube Music Premium status instead of the "Get Music Premium" button from the account menu. Modified code is based on /src/renderer.ts#L337-L355.

@JellyBrick JellyBrick added the bug Something isn't working label Feb 16, 2025
@JellyBrick
Copy link
Collaborator

JellyBrick commented Feb 16, 2025

What happens if the user enabled the "Remove upgrade button" feature?

@franz-dc
Copy link
Contributor Author

franz-dc commented Feb 16, 2025

What happens if the user enabled the "Remove upgrade button" feature?

It will still find the "Upgrade" button since the remove upgrade button feature only applies a display: none style instead of removing it from the DOM.

@JellyBrick JellyBrick merged commit 61ea104 into th-ch:master Feb 17, 2025
6 checks passed
@franz-dc franz-dc deleted the fix/downloader-premium-checker branch February 17, 2025 08:04
@Mendungo
Copy link

so do we have to sign in and get youtube premium?

@franz-dc
Copy link
Contributor Author

so do we have to sign in and get youtube premium?

You need to sign in, but you don't need Premium.

@Mendungo
Copy link

is signing in a risk for account suspension? i already got suspended once

@franz-dc
Copy link
Contributor Author

is signing in a risk for account suspension? i already got suspended once

Afaik you won't, but if you're really skeptical just make a new, separate one.

Did you get suspended because of using this app?

@Mendungo
Copy link

Yes for two days, that's when they put a block on the downloader of the software and the issue appeared

@Mendungo
Copy link

Mendungo commented Feb 20, 2025

two more things just so ya'll know, if you sign in and don't have premium songs will be downloaded in 128kbs even though it says 256kbs. Also for some reason it only downloads 100 songs if from download playlist selection even though there are 271 songs other than that it works great. Oh forgot to mention downloading playlist is a no go this is the error that pops up
Screenshot 2025-02-19 215620

@franz-dc
Copy link
Contributor Author

two more things just so ya'll know, if you sign in and don't have premium songs will be downloaded in 128kbs even though it says 256kbs. Also for some reason it only downloads 100 songs if from download playlist selection even though there are 271 songs other than that it works great. Oh forgot to mention downloading playlist is a no go this is the error that pops up !

This is related to another issue that still remains unresolved (#1791).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug]: (Downloader) YouTube Premium check fails if account menu is not opened at least once
3 participants