chore: Make /ping commands run in DMs only
#13
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
If a ping command is run in any other context, then an ephemeral message is sent to notify the user to use run the command in DMs.
The reason behind this change is to have users enable DM messaging between the bot and themselves to allow for Pi-Bot to send messages without 403ing. Previously, a user could set up pings, but unbeknownst to them, they would not be able to receive any pings due to a privacy limitation.
In a future PR, we will think of the option to change the
/pingcommand to be only visible within DMs. See #12Checks
blackover the code to ensure formatting.Important Info
pipdependencies.Issues Closed
My pull request closes the following issues:
N/A
Thank You
Thank you for your contribution to Scioly.org! This pull request will be reviewed
in a promptly manner. If not done, please feel free to contact @cbrxyz.