Skip to content

fix: correct 'inconvience' to 'inconvenience' typo in bot maintenance message - #56

Open
Jah-yee wants to merge 1 commit into
Valks-Bots:masterfrom
Jah-yee:fix/inconvience-to-inconvenience
Open

fix: correct 'inconvience' to 'inconvenience' typo in bot maintenance message#56
Jah-yee wants to merge 1 commit into
Valks-Bots:masterfrom
Jah-yee:fix/inconvience-to-inconvenience

Conversation

@Jah-yee

@Jah-yee Jah-yee commented Aug 1, 2026

Copy link
Copy Markdown

Summary

Corrects the typo inconvienceinconvenience in the bot maintenance error message.

Files Changed

  • src/events/message.js (line 31): user-facing error message shown when bot is in maintenance mode
  • docs/events_message.js.html: HTML documentation (generated, updated for consistency)

Before

client.embed.send(message, { desc: '... Sorry for the inconvience.' })

After

client.embed.send(message, { desc: '... Sorry for the inconvenience.' })

Why

This is a user-facing error message displayed when the bot is in maintenance mode. The typo reduces the perceived quality of the bot.

@surmado-code-review

Copy link
Copy Markdown

Automated Checks (advisory, non-blocking)

✅ All checks passed.


Summary

This PR makes a very small, low-risk copy fix in the maintenance-mode response in src/events/message.js, correcting a user-facing typo from “inconvience” to “inconvenience.” It also updates the corresponding generated HTML doc file for consistency. Reviewer: most of the risk is just confirming docs/events_message.js.html is expected to be committed/generated — the runtime change itself is straightforward.

What to pay attention to

  • src/events/message.js: This is a direct user-facing message change on the maintenance-mode path, so the main check is simply that the wording update is correct and doesn’t alter behavior.
  • docs/events_message.js.html: If this file is generated, the reviewer may want to confirm the repo convention is to commit generated docs updates alongside source changes.

Things I noticed

No red or yellow flags visible in this diff. This appears to be an isolated copy fix with no logic or error-handling impact.

Good patterns

  • The runtime string in src/events/message.js was updated directly where users will see it.
  • The matching documentation artifact was kept in sync with the source change.

💡 No engineering standards found. Surmado Code Review works best with a .github/STANDARDS.md that defines your team's conventions. Visit app.surmado.com to create one with the standards assistant — it takes about 5 minutes.


Surmado Code Review (v1.2-mt, review #1 on 78e0b14) is an automated review, designed to work alongside human judgment.

Want to change your STANDARDS.md or YML? Edit it directly, or tune it with our AI agent Scout.

Comment /rerun-review on this PR to re-review the current code with your latest STANDARDS.md and config — costs 1 additional PR credit (limit 3/hour per PR).

@Jah-yee

Jah-yee commented Aug 8, 2026

Copy link
Copy Markdown
Author

Friendly ping — just checking on this typo fix. Happy to address any feedback!

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