AppleSkin icon indicating copy to clipboard operation
AppleSkin copied to clipboard

Modrinth doesn't allow client side: `required` with server side: `optional`

Open dhouck opened this issue 1 year ago • 4 comments

Currently, Modrinth reports that this mod is unsupported on servers. Unless there have been some big changes since I last checked, this is incorrect; the Modrinth metadata should be updated to say itʼs optional for servers and required for clients.

(Alternately, if the metadata is correct, the description is out of date as it recommends installing on the server too)

Edit: add screenshot image

dhouck avatar Feb 06 '24 02:02 dhouck

Thanks, fixed. Not sure what happened as it was set to optional before, see https://github.com/squeek502/AppleSkin/issues/275

squeek502 avatar Feb 06 '24 23:02 squeek502

Iʼm still seeing it as unsupported; is that sometimes slow to propagate, or will it only take effect with a new version?

dhouck avatar Feb 07 '24 02:02 dhouck

Seems like a Modrinth bug, it keeps resetting to "Unsupported" somehow.

squeek502 avatar Feb 07 '24 10:02 squeek502

~~Set both client and server side to required for now since~~ Modrinth seems to only allow certain combinations and 'client required + server optional' doesn't seem to be an allowed combination.

The relevant issue is https://github.com/modrinth/labrinth/issues/883

EDIT: Actually, re-set it as 'unsupported' on the server-side to avoid the scenario in https://github.com/squeek502/AppleSkin/issues/275

EDIT#2: On third thought, I think setting client-side and server-side to 'optional' might make the most sense, so I did that.

Will need a fix on Modrinth's end to get this set properly.

squeek502 avatar May 01 '24 05:05 squeek502