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 the VNDB API as a Game source for visual novels #165

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

Senyksia
Copy link

@Senyksia Senyksia commented Sep 17, 2024

This adds VNDB as a new API source for visual novels. The API documentation and endpoint can be found here. All GameModel fields are populated, and the SFW flag is supported1.

Spurred primarily by #164, and Steam typically having a blind spot for visual novels.

Some implementation details for RFC:

  • I've set the number of results requested per search query to 20 (from the default 10), which I believe matches our other sources.
  • The GameModel.genres field is populated with content tags that have been scored as significant, and aren't spoilers. This is my interpretation of what most people will actually want, but we can always spit out every tag.
  • Because VNDB can return a partial release date (e.g. 2024-10), the plugin dateFormatter will end up coercing these dates to an erroneous specificity (e.g. 01-10-2024). This is relatively minor.

Footnotes

  1. VN entries with no official SFW releases and no content toggle are filtered out.

Previously entries with any NSFW releases would be filtered.
This has been relaxed to allow entries with either one standalone
SFW release, zero standalone NSFW releases, or the
`In-game Sexual Content Toggle` tag.

The `GameModel.image` field is now rudimentarily expunged of sensitive
images when the SFW filter is enabled. This is not ideal, but likely
requires `GameModel` changes to address properly.
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.

1 participant