Closed myssto closed 4 days ago
@cytusine0 @SourMongoose @hburn7 Feel free to edit or comment with additional thoughts
I think this flow is good. An admin can scan all of the pre-rejected / pre-verified entities, manually reject entities as needed, then click a button which verifies all pre-verified and rejects all pre-rejected. I considered having these be separate actions but I like the combined action better.
Now that the base functionality of the admin panel is complete, we should start implementing some custom functions that will improve productivity and reduce the amount of clicks it will take to manually review a tournament submission. All routes detailed here should be implemented on the
EntityController.Admin.cs
partial class.POST
/tournaments/{id}:accept-verification-status
ProcessingStatus
ofNeedsVerification
VerificationStatus
ofPreVerified
toVerified
andPreRejected
toRejected
. Functionally this can be seen as "accepting" the outcome of automation checks. https://github.com/osu-tournament-rating/otr-api/pull/521POST
/tournaments/{id}:rerun-auto-checks
ProcessingStatus
bool force = false
force = false
skip any entities with a concrete verification (Verified
orRejected
), else do not skip anyVerificationStatus
toNone
RejectionReason
toNone
WarningFlags
toNone
(if applicable)ProcessingStatus
toNeedsAutomationChecks
#522DELETE
to delete an entireTournament
https://github.com/osu-tournament-rating/otr-api/pull/506DELETE
endpoint to delete aMatch
https://github.com/osu-tournament-rating/otr-api/pull/507DELETE
endpoint to delete aGameScore
https://github.com/osu-tournament-rating/otr-api/pull/509(This is a draft, more should be added and then singular issues created for each endpoint once we determine the features we want)