mirror of
https://github.com/spotipy-dev/spotipy.git
synced 2026-04-27 00:25:54 +03:00
[GH-ISSUE #430] AttributeError: 'str' object has no attribute 'get_cached_token' #254
Labels
No labels
api-bug
bug
dependencies
documentation
duplicate
enhancement
external-ide
headless-mode
implicit-grant-flow
invalid
missing-endpoint
pr-welcome
private-api
pull-request
question
spotipy3
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/spotipy#254
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @ritiek on GitHub (Jan 23, 2020).
Original GitHub issue: https://github.com/spotipy-dev/spotipy/issues/430
The recent merge of #428 (
87a72a060f) seems to be breaking this example code:with this error:
The code works OK for commits earlier than
87a72a060f.This needs to be checked.
@stephanebruckert commented on GitHub (Jan 23, 2020):
Sorry about this, I've temporarily reverted that PR
github.com/plamere/spotipy@26db832e24and will have a better look at it tonight@ritiek commented on GitHub (Jan 23, 2020):
Our test suite should've caught this one. I'll also take a look right now what went wrong in #428. Adding automated tests for this later should be a good idea too!
@stephanebruckert commented on GitHub (Jan 23, 2020):
@ritiek I found what went wrong. Before merging, the tests were passing locally but I ran them from the
/testsfolder rather than the project folder, which would have used my pip-installed version of spotipy (2.7.1) instead of the project code. Will add a github action for tests as soon as this is fixed@MaZderMind commented on GitHub (Jan 26, 2020):
A way to still allow this kind of api usage might be, to check the type of the
auth=property and use it als-is when it is string.@stephanebruckert commented on GitHub (Jan 26, 2020):
@MaZderMind, @ritiek please let me know what you think of https://github.com/plamere/spotipy/issues/263#issuecomment-578539713
I'm closing the current issue since the breaking PR was reverted