-
Notifications
You must be signed in to change notification settings - Fork 602
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
fix(downloader): use the upgrade button to check for premium status #2987
Conversation
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 |
so do we have to sign in and get youtube premium? |
You need to sign in, but you don't need Premium. |
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? |
Yes for two days, that's when they put a block on the downloader of the software and the issue appeared |
This is related to another issue that still remains unresolved (#1791). |
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.