Skip to content

Update discord.mdx - #596

Closed
NFTDreww wants to merge 2 commits into
security-alliance:docs/normalize-community-managementfrom
NFTDreww:patch-7
Closed

Update discord.mdx#596
NFTDreww wants to merge 2 commits into
security-alliance:docs/normalize-community-managementfrom
NFTDreww:patch-7

Conversation

@NFTDreww

@NFTDreww NFTDreww commented Aug 4, 2026

Copy link
Copy Markdown
Collaborator

Added some additional context, review and hyperlinks for proper links!

What does this PR change?

Type of change

  • New content
  • Edit to existing content
  • Outline / structure change
  • Typo or formatting fix
  • Tooling / config

If applicable

  • Editing existing content: tagged the current contributors from the attribution list
  • Framework has a steward: asked them to review
  • Outline change: updated vocs.config.ts with the dev: true parameter
  • Want community feedback: shared this PR in our Discord

Stuck on anything? Just write it here and we're happy to help.

Added some additional context, review and hyperlinks for proper links!
@NFTDreww

NFTDreww commented Aug 4, 2026

Copy link
Copy Markdown
Collaborator Author

Update some language, Added hyperlinks to direct mentions of bots to their actual website, and update some wording to get away from whitelist -> allowlist.

@github-actions

github-actions Bot commented Aug 4, 2026

Copy link
Copy Markdown
built with Refined Cloudflare Pages Action

⚡ Cloudflare Pages Deployment

Name Status Preview Last Commit
frameworks ✅ Ready (View Log) Visit Preview a914706

@frameworks-volunteer frameworks-volunteer left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

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:

  1. Typo: "succesfull" → "successful" (line in Moderation section)

  2. 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."

  3. 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.

  4. 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.

  5. 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.
@scode2277
scode2277 deleted the branch security-alliance:docs/normalize-community-management August 6, 2026 15:55
@scode2277 scode2277 closed this Aug 6, 2026
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