Update discord.mdx - #596
Conversation
Added some additional context, review and hyperlinks for proper links!
|
Update some language, Added hyperlinks to direct mentions of bots to their actual website, and update some wording to get away from whitelist -> allowlist. |
built with Refined Cloudflare Pages Action⚡ Cloudflare Pages Deployment
|
frameworks-volunteer
left a comment
There was a problem hiding this comment.
Model: z-ai/glm-5.2 Reasoning: medium Provider: openrouter
Review of PR #596 — Update discord.mdx
Security: No issues. All external links point to legitimate Discord bot sites, no secrets or injection vectors.
QA — request changes for the following:
-
Typo: "succesfull" → "successful" (line in Moderation section)
-
Broken grammar in the Moderation section sentence: "Elevated permissions can be gained via a succesfull 2FA Session via these bots and only allows approved links are allowed to be sent in a discord server." This has a double "via", subject-verb disagreement ("allows...are allowed"), and unclear meaning. Consider rewriting, e.g.: "These bots enforce link allowlisting by requiring a successful 2FA session before elevated permissions are granted. Only approved links can be sent in the Discord server."
-
Heading change "Moderation & Anti-Link (Allowlisting link Bots)" is awkward — inconsistent capitalization ("link" vs "Link") and doesn't parse cleanly. Consider: "Moderation & Link Allowlisting Bots" to match the original style.
-
Wick Bot is now listed in both Anti-Raid and Anti-Nuke sections. If it covers both categories that's fine, but consider clarifying or differentiating the use case in each section.
-
Minor: "discord server" should be "Discord server" (capitalized) for consistency with the rest of the page.
The hyperlink additions and bot recommendations are good improvements — just needs a copy-edit pass before merge.
Updated All Comments to adjust grammar, word choice, and hyperlinks.
Added some additional context, review and hyperlinks for proper links!
What does this PR change?
Type of change
If applicable
vocs.config.tswith thedev: trueparameterStuck on anything? Just write it here and we're happy to help.