-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
Modrinth Game Versions badge doesn't fetch all game versions #10493
Comments
I don't think I agree with this. |
I agree that, in most cases, you want only the versions compatible with the latest versions. However, in minecraft, you have to release a different package for the different minecraft versions. A mod might get a feature update that is ported to multiple minecraft versions with multiple packages. Consider the following procedure (top to bottom):
After that, the badge would only show the minecraft version 1.19 even though the newest version (v1.2) also supports 1.20 which is arguably more important, since most people play on the most recent version of minecraft. Therefore, the optimal solution would be, to only show the game versions of the most resent version v1.2 which, in the example case above, includes the two most recent versions. Since there are no waterproof ways to figure out the latest (feature) version of the mod (different authors version their mods differently, the only version not controlled by the author is the game version), I would prefer showing all the versions (or limiting the badge to showing only the latest five versions as this is the maximum anyways). |
OK. Thanks for the extra context. @PyvesB - as our resident minecraft expert, could I tag you in on this one? Also, @Ceedrich - could you give us an example of a modrinth Project ID or two that has this issue so we can see what the API responses look like? Another thing I will note here is: Because of the solution we adopted in #10346 this doesn't deal particularly well with really long lists of versions. Once there are more than 5 versions in the list, some data is missing. |
The issue is present in some of the most popular mods: Sodium link to Endpoint Concerning #10346, unfortunately there is no consistent way to version numbering when it comes to minecraft mods. Many use a pattern similar to However, Modrinth lets authors feature certain versions. A solution could be to filter the response by this featured key. That way, all the relevant (based on the author’s opinion) versions are considered when showing the badge. Edit: Modrinth features certain versions automatically, so this solution would work even for authors that do not bother deciding which versions to feature.
from https://support.modrinth.com/en/articles/8793360-version-auto-featuring |
OK. I do now understand the issue. Featured versions suggestion sounds like it could be reasonable. I think I'll see if P-Y has any thoughts on this one before proceeding. Thanks |
Out of interest, how are these featured versions rendered in the Modrinth UI? |
Couple more thoughts on this:
|
Are you experiencing an issue with...
shields.io
🐞 Description
When using the Modrinth Game Versions badge, you would expect to get all supported game versions, but you only get the ones from the latest release
🔗 Link to the badge
https://shields.io/badges/modrinth-game-versions
💡 Possible Solution
The fix would be relatively simple.
Only the function
async handle({ projectId })
inservices/modrinth/modrinth-game-versions.service.js
would need to change from:to:
The text was updated successfully, but these errors were encountered: