Skip to content

Upgrade Beginner and coding command. Added check for suggestion.cpp - #41

Open
Gag91 wants to merge 1 commit into
cppdiscord:mainfrom
Gag91:main
Open

Upgrade Beginner and coding command. Added check for suggestion.cpp#41
Gag91 wants to merge 1 commit into
cppdiscord:mainfrom
Gag91:main

Conversation

@Gag91

@Gag91 Gag91 commented Aug 15, 2026

Copy link
Copy Markdown

Upgraded

Beginner Command

  • Clickable Title
  • Added Buttons
  • Better code structure
  • Uses globals::channel::HELP_CHANNEL_ID from globals.h

Coding Command

  • Difficulty files
  • Difficulty option

globals.h

  • Added namespace channel
  • Added HELP_CHANNEL_ID to namespace channel

commands.h

  • Modified std::list to std::vector

suggestion.cpp | Checks

Delete suggestion

  • Check if message has embeds
  • Check if embed has author
  • Verify clicker is the original author
  • Show ephemeral error if not author
  • Delete message if author

Edit suggestion

  • Check if message has embeds
  • Check if embed has author
  • Verify clicker is the original author
  • Show ephemeral error if not author
  • Open modal if author

Edit modal submission

  • Check if components exist
  • Check if component data exists
  • Try-catch for value extraction
  • Check if text is empty
  • Check if original message has embed
  • Check if message fetch succeeded
  • Check if fetched message has embed
  • Show ephemeral error for each failure
  • Show success confirmation

Tests

  • It Compiles

  • It Works

@transicle

Copy link
Copy Markdown
Contributor

Is your PR message AI generated, lol? Your English typically isn't this good.

@transicle

Copy link
Copy Markdown
Contributor

@ebcq If considering merging, take into account some of these files were generated 100% by AI.

@Gag91

Gag91 commented Aug 16, 2026

Copy link
Copy Markdown
Author

Note

  • Nothing was vibecoded
  • The text files in res/coding for the coding command were AI generated
  • I do not see any issue using AI to generate a list of question
  • This pull request satisfies an approved suggestion

Discord

  • Username: xxavi640
  • I am on the C++ server

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