Skip to content

Conversation

@evopls
Copy link
Contributor

@evopls evopls commented Sep 4, 2025

Changed redirect_uri to work with new Spotify guidelines - readme still needs to be updated to reflect the changes.

#31


Important

Update redirect_uri in auth() in main.py to comply with new Spotify guidelines.

  • Behavior:
    • Update redirect_uri in auth() in main.py from http://localhost:8088 to http://127.0.0.1:8088 to comply with new Spotify guidelines.
  • Documentation:
    • README update is pending to reflect this change.

This description was created by Ellipsis for 1da1eae. You can customize this summary. It will automatically update as commits are pushed.

Changed `redirect_uri` to work with new Spotify guidelines - readme still needs to be updated to reflect the changes.
Copy link

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Important

Looks good to me! 👍

Reviewed everything up to 1da1eae in 26 seconds. Click for details.
  • Reviewed 13 lines of code in 1 files
  • Skipped 0 files when reviewing.
  • Skipped posting 1 draft comments. View those below.
  • Modify your settings and rules to customize what types of comments Ellipsis leaves. And don't forget to react with 👍 or 👎 to teach Ellipsis.
1. aw_watcher_spotify/main.py:70
  • Draft comment:
    Changed redirect_uri to 127.0.0.1 as per Spotify's new guidelines. Ensure the Spotify developer settings and README reflect this update, and consider parameterizing the URI if a different port is needed for local testing.
  • Reason this comment was not posted:
    Confidence changes required: 0% <= threshold 50% None

Workflow ID: wflow_S2NSh93rI15KFocj

You can customize Ellipsis by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.

evopls added a commit to evopls/aw-watcher-spotify that referenced this pull request Sep 4, 2025
@ErikBjare ErikBjare changed the title change redirect_uri to work with new spotify guidelines fix: change redirect_uri to work with new spotify guidelines Sep 8, 2025
@ErikBjare ErikBjare merged commit 148f10b into ActivityWatch:master Sep 8, 2025
1 check failed
ErikBjare pushed a commit that referenced this pull request Sep 8, 2025
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.

2 participants