Skip to content
This repository has been archived by the owner on Oct 19, 2024. It is now read-only.

Loader + game version combo search field #622

Closed
wants to merge 1 commit into from

Conversation

stairman06
Copy link
Contributor

Required backend implementation for modrinth/code#1907.

My SQL might be poor so feel free to critique.

Copy link
Member

@Geometrically Geometrically left a comment

Choose a reason for hiding this comment

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

why not just use the existing joins for this?

@stairman06
Copy link
Contributor Author

I don't believe this is possible with the existing joins? They don't have the right association between version and game version/loader. But I'll need to check and look at it again.

@Geometrically
Copy link
Member

couldn't you only join the combos and then extract the rest of the versions and loaders in the rust side of code?

@stairman06
Copy link
Contributor Author

I think with that we'd end up with the loader ID and the game version ID, which aren't what we want. To actually convert those to the loader name and game version name we have to ask the database AFAIK. But maybe I'm misunderstanding?

@Geometrically
Copy link
Member

Superseded by modrinth/code#1778

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants