leaderboardsgg / leaderboard-backend

The backend for Leaderboards.gg. Also holds public-facing APIs.
https://info.leaderboards.gg
GNU General Public License v3.0
12 stars 16 forks source link

Task: Swap FluentValidation for Pure Data Annotations #247

Open zysim opened 1 month ago

zysim commented 1 month ago

We probably don't need FluentValidation for this project. Data annotations should handle our use cases already, plus we have to add extra, misleading annotations to handle null request field validation. Specifically; we need to annotate [Required(AllowEmptyStrings = true)] in order to fix what the Swagger doc says, even though we don't actually allow empty strings for that request.

Affected Functionality