Skip to content

Fix redirect URI formatting in README after recent changes#106

Merged
danielfm merged 3 commits intodanielfm:masterfrom
JoaoZPalma:patch-1
Dec 19, 2025
Merged

Fix redirect URI formatting in README after recent changes#106
danielfm merged 3 commits intodanielfm:masterfrom
JoaoZPalma:patch-1

Conversation

@JoaoZPalma
Copy link
Contributor

Correct the redirect URI format in the README.

Correct the redirect URI format in the README.
@JoaoZPalma
Copy link
Contributor Author

JoaoZPalma commented Oct 12, 2025

Was all happy cuz i got the Spotify to connect to Emacs but now its giving me:

Log

smudge connected
url-retrieve-internal: Bad url: d3a97b8e006a4eb98fffe9c2bdecaa1a
Error running timer ‘smudge-controller-player-status’: (void-function smudge-connect-player-status) [2 times]

Not from my change, mine its merely the ReadMe

@girishmm
Copy link

This has been mentioned in #105. Would be nice if you can add this to your PR :)

Also, to make the URI fully compatible with new Spotify regulations, localhost must be changed to 127.0.0.1 as mentioned in #101 and #102

@JoaoZPalma
Copy link
Contributor Author

JoaoZPalma commented Oct 16, 2025

Thank you @girishmm, I changed localhost to 127.0.0.1 locally but completly went over my head at the ReadMe, also added that change in #105

Didn't have time to test it out tho, whoever is checking out this pr please take it with a grain of salt

@JoaoZPalma JoaoZPalma closed this Oct 16, 2025
@JoaoZPalma JoaoZPalma reopened this Oct 16, 2025
@danielfm
Copy link
Owner

Thanks!

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.

3 participants