Skip to content

Update Discord Guide to match Discord's settings redesign - #613

Merged
scode2277 merged 9 commits into
security-alliance:developfrom
NFTDreww:patch-7
Sep 8, 2026
Merged

Update Discord Guide to match Discord's settings redesign#613
scode2277 merged 9 commits into
security-alliance:developfrom
NFTDreww:patch-7

Conversation

@NFTDreww

@NFTDreww NFTDreww commented Aug 27, 2026 •
edited by scode2277
Loading

Copy link
Copy Markdown
Collaborator

What does this PR change?

Refreshes the Discord account management guide so it matches Discord's current UI, and tightens the security bots section.

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.

mattaereal and others added 5 commits July 26, 2026 20:49
@mattaereal
docs(community-management): normalize structure and wording
63568b9 
Add Key Takeaways, canonical page map, related frameworks, Further reading,
and markdownlint-clean tables/line wraps. Preserve existing contributors and
guide deep-link structure; no new security controls.
@mattaereal
docs: clamp descriptions to 120–170 content band
8bdcb81 
Rewrite non-stub frontmatter descriptions into the acceptable length
band (preferred 140–160). Stub pages exempt. No body or credit changes.
@mattaereal
docs: align further-reading headings with validate:content
8b62246 
Rename Further Reading H2s to the exact title the updated security-alliance#561 validator accepts; sentence-case a few common section headings.
@NFTDreww
Update discord.mdx
86373ac 
Added some additional context, review and hyperlinks for proper links!
@NFTDreww
Update discord.mdx
a914706 
Updated All Comments to adjust grammar, word choice, and hyperlinks.
@NFTDreww

NFTDreww commented Aug 27, 2026

Copy link
Copy Markdown
Collaborator Author

Re-Opening this for merge due to me being tired and trying to merge to wrong branch lol

#596

@NFTDreww NFTDreww mentioned this pull request Aug 27, 2026
Closed
9 tasks
frameworks-volunteer
frameworks-volunteer suggested changes Aug 31, 2026
View reviewed changes

@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: x-ai/grok-4.5 Reasoning: high Provider: openrouter

Review: PR #613 — Updating Discord Guide

Verdict: REQUEST CHANGES

Security: no secrets, no executable risk, docs-only. CI preview build passed. External bot URLs largely resolve (some hosts return 403 to bots; expected).

Blocking

  1. Merge conflicts with current develop (mergeable_state: dirty)

    • ## Further Reading vs develop’s ## Further reading on discord/overview/telegram/twitter hubs
    • community-management/overview.mdx description and platform list: develop already includes Matrix / Session / SimpleX (#607). This branch still describes only Discord / X / Telegram and would drop those entries if conflict-resolved the wrong way
    • Rebase onto upstream/develop and keep the six-platform list + sentence-case ## Further reading

  2. Factually wrong bot category text — docs/pages/guides/account-management/discord.mdx (Moderation & Link Allowlisting)

    • Current text mixes two unrelated controls: “require a successful 2FA session before elevated permissions” and “only allow approved links”
    • Link allowlisting ≠ 2FA-gated privilege elevation. Split or rewrite so each bot matches what it actually does
    • ZTS Guard is also listed again under General Moderation without explaining the dual role

  3. Wick placement inconsistency

    • Hub topic index still points anti-impersonation at Wick
    • Guide body now puts Wick under Anti-Nuke
    • Hashbot remains the anti-impersonation example in the guide
    • Align hub index + guide categories (or say Wick covers multiple roles with clear bullets)

Should fix

  1. Stable Checklist id churn — step-2-create-the-discord-account → step-2-create-the-Discord-account resets any persisted checklist state. Keep the original id; only change display text if needed

  2. Hostname capitalization noise — https://Discord.com/... and support-dev.Discord.com add no value and diverge from existing lowercase Discord URLs. Prefer canonical lowercase hosts

  3. PR hygiene

    • Title/body still the empty template; title says Discord only but the diff also rewrites overview/telegram/twitter hubs
    • Fill type-of-change + short summary of intent (bot links + wording vs structural hub rewrite)

Looks good

  • Useful official/product links for Beemo, Wick, Dyno, Carl-bot, Goodknight
  • “Whitelisting” → “Allowlisting” wording
  • Capitalizing product name “Discord” in prose (separate from URL hosts)
  • Preview site build green

Rebase, fix the allowlisting/2FA conflation and Wick consistency, restore overview multi-platform coverage, then ping for re-review.

@NFTDreww
Update discord.mdx
31346bb 
I have performed the below actions based on feedback:

2. I have split out the section for moderation and elevated permissions and then anti-link so this is more clear and understand for the bot categories

3. Easiest to add additional bullet points for wick as I did since it is a multiple purpose bot

4. & 5. this was adjusted fixed and done, all lowercase

6. Noted and heard

I have also implemented some wording, fixes, and where certain features are now within the Discord platform as of their latest update on 08/21 which changed the flow of getting to settings.  This mainly affected lines within 70 - 90
@NFTDreww

NFTDreww commented Aug 31, 2026

Copy link
Copy Markdown
Collaborator Author

Added updated Commity:

I have performed the below actions based on feedback:

  1. I have split out the section for moderation and elevated permissions and then anti-link so this is more clear and understand for the bot categories

  2. Easiest to add additional bullet points for wick as I did since it is a multiple purpose bot

  3. & 5. this was adjusted fixed and done, all lowercase

  4. Noted and heard

I have also implemented some wording, fixes, and where certain features are now within the Discord platform as of their latest update on 08/21 which changed the flow of getting to settings. This mainly affected lines within 70 - 90

@NFTDreww

NFTDreww commented Sep 7, 2026

Copy link
Copy Markdown
Collaborator Author

@scode2277 This one actually is the right one that I merged everything into, so ignore #611

@github-actions

github-actions Bot commented Sep 8, 2026 •
edited
Loading

Copy link
Copy Markdown
built with Refined Cloudflare Pages Action

⚡ Cloudflare Pages Deployment

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

@scode2277 scode2277 changed the title Updating Discord Guide Update Discord Guide to match Discord's settings redesign Sep 8, 2026
@scode2277
scode2277 merged commit 4a2b878 into security-alliance:develop Sep 8, 2026
5 of 6 checks passed
@scode2277 scode2277 added the content:update This issue or PR updates content or suggests to label Sep 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@scode2277 scode2277 scode2277 approved these changes
+1 more reviewer
@frameworks-volunteer frameworks-volunteer frameworks-volunteer requested changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

content:update This issue or PR updates content or suggests to

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@NFTDreww @scode2277 @frameworks-volunteer @mattaereal