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 ListenBrainz implementation #258

Merged
merged 9 commits into from
Jul 31, 2023
Merged

Conversation

Mar0xy
Copy link
Contributor

@Mar0xy Mar0xy commented Jul 30, 2023

This should close #253

I also noticed that the two lines

skippedArtists.value = settingsStore.get<string, string[]>(settings.skippedArtists).join("\n");
customCSS.value = settingsStore.get<string, string[]>(settings.customCSS).join("\n");

in refreshSettings seem to break the settings window as it can't run the refresh function due to those two lines throwing a .join() is not a function error I didn't fix this error but I thought I would let you know as I was also confused as to why my normal install of tidal-hifi does not show my actual settings and defaults to everything being off and empty.

@Mar0xy
Copy link
Contributor Author

Mar0xy commented Jul 30, 2023

Turns out the error that I mentioned was caused by the fact that customCSS for some reason was set to "" in my config instead of []

@Mastermindzh Mastermindzh changed the base branch from master to 5.5.0 July 31, 2023 08:53
@sonarcloud
Copy link

sonarcloud bot commented Jul 31, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@Mastermindzh Mastermindzh merged commit ec82aa8 into Mastermindzh:5.5.0 Jul 31, 2023
5 checks passed
@Mar0xy Mar0xy deleted the master2 branch July 31, 2023 17:25
@Mar0xy Mar0xy mentioned this pull request Aug 2, 2023
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.

Direct scrobbling implementation for ListenBrainz
2 participants