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

Use Windows SID instead of the name Everyone for better system language compatibility #47

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

dionysius
Copy link

See #46 for all details

@dionysius
Copy link
Author

dionysius commented May 24, 2021

I will run into this issue again and again whenever there is a new spotify update on scoop as long this is not merged. Any non-english system languague will have issues with the internal updater.

By the way, the bug is visible. If you look back at my comment: #34 (comment), this part indicates the error:

Everyone: No mapping between account names and security IDs was done.

What do you need from me to get this merged?

karthikvinayan added a commit to karthikvinayan/Scoop-Spotify that referenced this pull request Oct 30, 2021
…ge compatibility

	modified:   bucket/spotify-latest.json
	credits
	TheRandomLabs#47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant