Skip to content

feat: add project reviews system#6118

Open
chank-op wants to merge 8 commits into
modrinth:mainfrom
chank-op:main
Open

feat: add project reviews system#6118
chank-op wants to merge 8 commits into
modrinth:mainfrom
chank-op:main

Conversation

@chank-op
Copy link
Copy Markdown
Contributor

G'day

Summary

  • Adds a reviews system for Modrinth projects — users can leave a star rating (1–5) and optional text review per project
  • Backend: new project_reviews table, full CRUD API under /v3/review(s) with scoped auth (REVIEW_CREATE/READ/WRITE/DELETE)
  • Frontend (web + app): new /reviews tab on project pages using TanStack Query for fetching, submitting, editing, and deleting reviews
  • API client: new LabrinthReviewsV3Module registered as client.labrinth.reviews_v3

implementing: #6095

chank-op and others added 6 commits May 10, 2026 18:00
- JAR files were downloading with a `.zip` extension in Chromium-based browsers (Chrome, Edge, Arc, Brave, Opera, Vivaldi)
- Root cause: JAR files are ZIP archives internally, so Chromium sniffs the `Content-Type` as `application/zip` and overrides the filename extension when no `download` attribute is present
- Fix: add `download="<filename>"` to all file download `<a>` tags so the browser uses the original filename from the API
Missed in initial pass: changelog page button, versions overflow
menu, settings/versions overflow menu. Also adds `download` prop
to Button and OverflowMenu to support dropdown link items.

Adds missing `getPrimaryFile` definition in changelog.vue.
@modrinth-bot
Copy link
Copy Markdown
Member

Pull request changelog

App

Added

Changed

Deprecated

Removed

Fixed

Security

Website

Added

Changed

Deprecated

Removed

Fixed

Security

Hosting

Added

Changed

Deprecated

Removed

Fixed

Security

chank-op added 2 commits May 17, 2026 10:16
Accidentally gitignored and deleted .sqlx/ in previous commit, breaking
CI which requires SQLX_OFFLINE=true to compile without a live database.
Regenerated all 574 query cache files via `cargo sqlx prepare`.
@chank-op
Copy link
Copy Markdown
Contributor Author

one of my commits accidentally gitignored and deleted the .sqlx offline query cache, which would have broken CI compilation. Fixed in the latest commit f89008e all query cache files restored and .sqlx/ removed from .gitignore.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants