Skip to content

Include mod in action outputs in the thread#67

Open
hyperneutrino wants to merge 1 commit into
mainfrom
hyper/callout-mod-in-thread
Open

Include mod in action outputs in the thread#67
hyperneutrino wants to merge 1 commit into
mainfrom
hyper/callout-mod-in-thread

Conversation

@hyperneutrino
Copy link
Copy Markdown
Member

Right now, figuring out which mod approved/denied a verification and looking at the verification itself requires finding two separate pieces of information. We can streamline this a bit by simply outputting the acting mod in the ticket.

Note that this message is generated after the action is taken so users being approved will be locked out by receiving the verified role and denied users will be removed from the server first. That said, showing the acting mod to approved users isn't much of a big deal as usually it's the person asking them questions and they will likely assume that anyway.

@hyperneutrino
Copy link
Copy Markdown
Member Author

Tested locally in DevPlace!

@MysticMia
Copy link
Copy Markdown
Contributor

Wouldn't user mentions be annoying as verifier? Could make it send the username and id of the verifier or make it a silent ping instead?
or maybe pings aren't as big of a deal because the verifier remains in the verify channel for the second that the message is sent.

@hyperneutrino
Copy link
Copy Markdown
Member Author

@MysticMia resolve includes allowedMentions: { parse: [] } in both instances that call verifyUser so it does not actually ping

@hyperneutrino hyperneutrino requested a review from MysticMia April 29, 2026 07:45
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.

2 participants