Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

More useful help and error messages #394

Merged
merged 2 commits into from
Jan 15, 2024
Merged

More useful help and error messages #394

merged 2 commits into from
Jan 15, 2024

Conversation

rebkwok
Copy link
Contributor

@rebkwok rebkwok commented Jan 11, 2024 •
edited
Loading

Some extra user help that came out of my show-and-tell:

  • add a description to each category of available slack commands to display in the main @BennettBot help
  • Label namespaces "categories" in the help message (Fixes Consider more user friendly alternative to "namespace" #34)
  • Post helpful error messages back to slack if an unexpected error is caught by the error handler

Base automatically changed from docs to main January 11, 2024 13:02
@rebkwok rebkwok force-pushed the more-help branch 3 times, most recently from 530e2e5 to 5a7c3e2 Compare January 11, 2024 16:55
@rebkwok
Report errors to slack for unexpected errors
74c0a76 
If something unexpected goes wrong while handling
a slack event, we have access to the original
request body, and can respond to the message and
report the error.
@rebkwok
Add a description to namespace/category help
05c5e58
@rebkwok rebkwok changed the title More help More useful help and error messages Jan 12, 2024
@madwort
Copy link
Contributor

madwort commented Jan 12, 2024

hah, does this close #34 ?

@rebkwok
Copy link
Contributor Author

rebkwok commented Jan 15, 2024

hah, does this close #34 ?

I think it does :)
Not for people making new job configs, but then if they're doing that, they probably know what namespace means anyway

@rebkwok rebkwok merged commit 03d619d into main Jan 15, 2024
6 checks passed
@rebkwok rebkwok deleted the more-help branch January 15, 2024 14:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@inglesp inglesp inglesp approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Consider more user friendly alternative to "namespace"
3 participants
@rebkwok @madwort @inglesp