Skip to content

Add private comments and anonymous actions - #165

Open
A6sT wants to merge 3 commits into
Quaver:mainfrom
A6sT:main
Open

Add private comments and anonymous actions#165
A6sT wants to merge 3 commits into
Quaver:mainfrom
A6sT:main

Conversation

@A6sT

@A6sT A6sT commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

Changes needed for new site feature, allowing RS to take action anonymously.

@A6sT A6sT closed this Aug 15, 2026
@A6sT

A6sT commented Aug 15, 2026

Copy link
Copy Markdown
Contributor Author

I got a bit confused on the branch i was supposed to open the pr on, woops

@A6sT A6sT reopened this Aug 15, 2026
@AiAe
AiAe requested a review from Swan August 15, 2026 21:19
Comment thread db/mapset_ranking_queue_comments.go Outdated
RankingQueueActionOnHold
RankingQueueActionVote
RankingQueueActionResolved
RankingQueueActionPrivate

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Curious why we should have a new action if we already have is_anonymous?

Wouldn't it make more sense to do RankingQueueActionComment and set is_anonymous to true and just use that?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@Swan we want it as new action, because when RS submits it, it wont take any real action to the ranking.
Its just to notify internally that the RS does not know what they wanna do with the mapset/does not know how to handle it.
Where if multiple of those are submitted then an HRS can deny the mapset.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Alternatively we can just name it as Comment instead Private + the option for it to be anonymous or public.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

we want it as new action, because when RS submits it, it wont take any real action to the ranking.

RankingQueueActionComment isn't a real action. It's just a comment. So having that with an anonymous flag should still give what you want - an anonymous comment.

Where if multiple of those are submitted then an HRS can deny the mapset.

Just check for multiple anonymous comments?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Fair forgot comments exists.
For now we want to try manually I think, we can make it automated eventually.

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.

3 participants