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

New ChirpRadio connector #4695

Closed
wants to merge 0 commits into from
Closed

Conversation

jbwharris
Copy link
Contributor

New connector for https://chirpradio.org

image

@jaccarmac jaccarmac added minor-change For minor changes new-connector For PRs that adds a new connector labels May 10, 2024
Copy link
Member

@inverse inverse left a comment

Choose a reason for hiding this comment

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

Looks good aside the additional changes in src/core/connectors.ts

/**
* true if website has its own scrobbling system the user needs to be aware of.
*/
hasNativeScrobbler?: true;
Copy link
Member

Choose a reason for hiding this comment

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

This file looks like it needs to be adapted to only the addition of the connector addition

@jbwharris
Copy link
Contributor Author

@inverse I just pushed over the change to connectors.ts

@inverse
Copy link
Member

inverse commented May 26, 2024

@jbwharris looks like there are still a lot of changes that shouldnt be there 🤔

@jbwharris
Copy link
Contributor Author

@inverse I have no idea what I've done wrong with this one. I'm going to create a new branch and do a new PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
minor-change For minor changes new-connector For PRs that adds a new connector
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants