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

Handle backend errors #14

Merged
merged 10 commits into from
Jul 5, 2023
Merged

Handle backend errors #14

merged 10 commits into from
Jul 5, 2023

Conversation

bkeys818
Copy link
Owner

@bkeys818 bkeys818 commented Jul 2, 2023

Description

These changes improve how errors are handled on the front and back end.

Progress

  • Clean up error message on the front-end
  • Improve backend error handling
  • Remove dependency 'spotify-web-api-node'

Related Errors

@bkeys818 bkeys818 added the bug Something isn't working label Jul 2, 2023
This was linked to issues Jul 2, 2023
@bkeys818
Copy link
Owner Author

bkeys818 commented Jul 4, 2023

Currently, I'm assuming that when a changePlaylistDetails request responds with a 403 error, the playlist has been deleted. According to the documentation, a 403 code might be returned for a handful of reasons.

To fix this, after receiving the 403 error, I'll check all the user's playlists to see if I can find one that matches. If not, then I'll delete the user's data.

@bkeys818 bkeys818 marked this pull request as ready for review July 5, 2023 16:06
@bkeys818 bkeys818 merged commit 7229114 into main Jul 5, 2023
3 checks passed
@bkeys818 bkeys818 deleted the handle-backend-errors branch July 5, 2023 16:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Error msg from "spotify-web-api-node" Unhandled backend errors
1 participant