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

fix: plugins/listenbrainz: Fix UnboundLocalError in cases where 'mbid' is not defined #5651

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

Conversation

allendema
Copy link

@allendema allendema commented Feb 27, 2025

Description

Fix ocurrence of UnboundLocalError in plugins/listenbrainz > get_tracks_from_listens() when mbid is not available.
Removed a print statment.
Fix link to config.yaml.
Fix link to Listenbrainz "get the token" documentation.

To Do

  • Documentation. (If you've added a new command-line flag, for example, find the appropriate page under docs/ to describe it.)
  • Changelog. (Add an entry to docs/changelog.rst to the bottom of one of the lists near the top of the document.)
  • Tests. (Very much encouraged but not strictly required.)

@allendema allendema force-pushed the fix-plugins-listenbrainz-unboundlocalerror branch from 41c9ea4 to 0c0d168 Compare February 27, 2025 18:59
@allendema allendema force-pushed the fix-plugins-listenbrainz-unboundlocalerror branch from 0c0d168 to 89c2f47 Compare February 27, 2025 19:07
@snejus
Copy link
Member

snejus commented Mar 3, 2025

Please add a PR description to link this to the issue and it should be good to go.

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