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

Add Stationhead connector #4068

Merged
merged 4 commits into from
Sep 1, 2023
Merged

Conversation

fikimaul
Copy link
Contributor

@fikimaul fikimaul commented Aug 29, 2023

Describe the changes you made
Stationhead now has web player. The web player can connect to apple music or spotify. With this connector user can scrobble when Stationhead connected to Apple Music or when lastfm not connected to spotify account.

The app url :
https://www.stationhead.com/ or https://app.stationhead.com/

Fix #2879

Additional context
Testing
image

Thank You

@fikimaul fikimaul changed the title Add Station Head connector Add Stationhead connector Aug 29, 2023
@inverse inverse added new-feature For PRs that add new functionality minor-change For minor changes labels Aug 29, 2023
export {};

Connector.useMediaSessionApi();
Connector.playerSelector = '#root>div>div:nth-child(2)>div>div>div>div>div:nth-child(2)>div:nth-child(2)>div';
Copy link
Member

Choose a reason for hiding this comment

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

Is there not any class or ID on the element to make this less brittle to changes?

Also please add a new line after :)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The website only have generated class from library they use, when website have an update will regenerate the class.
image

Done add new line on end of file.

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.

Don't have an account so didn't test but seems better than nothing.

Hopefully they don't change they layout too much as it seems quite brittle

@inverse inverse merged commit df511fd into web-scrobbler:master Sep 1, 2023
@fikimaul fikimaul deleted the add-stationhead branch September 2, 2023 11:23
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-feature For PRs that add new functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Requesting to add STATIONHEAD on scrobbling connector
2 participants