Skip to content

Improve error messages in codebuddy_quiz command#34

Open
shivani-d-13 wants to merge 1 commit intoTheCodeVerseHub:mainfrom
shivani-d-13:improve-error-messages
Open

Improve error messages in codebuddy_quiz command#34
shivani-d-13 wants to merge 1 commit intoTheCodeVerseHub:mainfrom
shivani-d-13:improve-error-messages

Conversation

@shivani-d-13
Copy link
Copy Markdown

@shivani-d-13 shivani-d-13 commented Mar 29, 2026

What changed?

Improved error handling in the codebuddy_quiz command by replacing vague error messages with clearer, user-friendly responses.

Why?

Link the issue (required for non-trivial changes):

How to test

  • Run the codebuddy_quiz command
  • Trigger an error scenario (e.g., invalid input or backend failure)
  • Verify that the error message shown is clear and user-friendly

Checklist

  • I linked an issue or explained why not
  • I added/updated docs for user-facing changes
  • I added/updated tests (or explained why not)
  • I did not commit secrets (tokens, .env, keys)
  • CI should pass for this PR

Screenshots (optional)

NA

@shivani-d-13 shivani-d-13 requested a review from a team as a code owner March 29, 2026 16:21
@youngcoder45
Copy link
Copy Markdown
Contributor

Kinda I like the PR but it sounds too minimal to be considered as a contribution idk what to do @TheCodeVerseHub/admin waiting for ur response

@shivani-d-13
Copy link
Copy Markdown
Author

Kinda I like the PR but it sounds too minimal to be considered as a contribution idk what to do @TheCodeVerseHub/admin waiting for ur response

Happy to expand this. Should I apply similar improvements across other files, or are there specific areas you'd like me to focus on?

@youngcoder45
Copy link
Copy Markdown
Contributor

I am unsure what to do.

Copy link
Copy Markdown
Contributor

@HyScript7 HyScript7 left a comment

Choose a reason for hiding this comment

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

The PR title is misleading. The only thing this PR does is rename a couple error messages and remove a leading space.

That said, it does make the error messages a tiny bit clearer, so I will be merging this.
Thank you for your contribution.

@HyScript7 HyScript7 changed the title fix: improve error handling and messages in codebuddy_quiz command Improve error messages in codebuddy_quiz command Apr 5, 2026
@HyScript7 HyScript7 requested a review from youngcoder45 April 5, 2026 19:06
Copy link
Copy Markdown
Contributor

@youngcoder45 youngcoder45 left a comment

Choose a reason for hiding this comment

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

Approved

print(f"[Unexpected error in codestats command]: {e}")
await ctx.send("Error displaying your stats.")
await ctx.send("Failed to load your stats. Please try again in a moment.")

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Since it counts as a contribution I approve this.

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