Skip to content

FEAT: final polish#25

Merged
Framebuffers merged 18 commits into
devfrom
dev-polishing
May 18, 2026
Merged

FEAT: final polish#25
Framebuffers merged 18 commits into
devfrom
dev-polishing

Conversation

@Framebuffers

Copy link
Copy Markdown
Owner

FEAT: command, UI/UX and security final polish

Done:

  • All hardcoded strings moved to messages.go
  • Session announcements: role mention (<@&roleID>), pinned thread messages, Unicode channel names (ñ,
    é, etc.), DD/MM/YYYY dates
  • Campaign archive/delete now hides the Discord channel from everyone except staff roles
  • welcome thread on campaign creation: locked, pinned, with a greeting message
  • Stale commands de-registered via ApplicationCommandBulkOverwrite (atomic, no manual cleanup)
  • /managecampaigns -> /manage with DM-campaign autocomplete
  • /campaignupload -> /uploadcover
  • /nextsessions command added (upcoming sessions, sorted, player timezone-aware)
  • /tokens command added (gallery entry point)
  • /ban, /unban, /admin hidden from /help
  • /ping removed
  • Download button on token creation (opaque one-time UUID link, 10-min TTL, internal path never
    exposed)
  • Campaign channel ID hidden from non-members in embeds
  • Flow header docs added to all new files

To Do

  • merge to main!!!!!!1!!

…er execution until all discord api calls return. if not, handle panic by responding with an ephemeral without crashing the whole bot
…ule conflict detection, capacity handling, DM notified for every RSVP
… is missing when it's a legacy DB or it hasn't been approved yet. Improved flow for pending campaigns: now they say when they're awaiting approval when checked, and they have access to the campaign menu now.
@Framebuffers Framebuffers merged commit 5f85a09 into dev May 18, 2026
1 check passed
@Framebuffers Framebuffers deleted the dev-polishing branch May 18, 2026 00:52
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.

1 participant