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

Possible maloja fork support #198

Open
duckfromdiscord opened this issue Oct 2, 2024 · 1 comment
Open

Possible maloja fork support #198

duckfromdiscord opened this issue Oct 2, 2024 · 1 comment
Labels
question Further information is requested

Comments

@duckfromdiscord
Copy link

duckfromdiscord commented Oct 2, 2024

I sent an email in August about this topic but it might've gone to spam.

I'm considering forking Maloja and adding some of my own features. One of the first features I add will be a new, separate spot for identifiers that all clients can use since it'll be a part of the API standard. There are some more ideas I have in mind which include a "now playing" feature that will require changes on multi-scrobbler's side, and I can pull request those if needed.

I spoke to a couple of other active Maloja users who have said they will use my fork if I make one. But if changes need to be made on multi-scrobbler's side, there will have to be a fork of that too, and it will be tough to maintain both. Ideally, we could add a config flag for your maloja client that enables features from my fork. It seems that including new parameters that my new API changes would take causes warnings from the server, so it would be best to include even just a hidden and off-by-default option to send these parameters alongside requests.

@FoxxMD
Copy link
Owner

FoxxMD commented Oct 2, 2024

MS already does a bit of "if maloja version is X or Y" due to supporting pre-3.0 versions.

compare-versions handles semver with tags just fine so my recommendation would be to use a tag in your fork's versionstring that MS could use to determine if it should send additional fields in the api call IE versionstring = '3.2.3-duck' or something.

@FoxxMD FoxxMD added the question Further information is requested label Dec 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants