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.

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

Make output of note commands private #112

Open
pmayd opened this issue Jul 28, 2023 · 2 comments
Open

Make output of note commands private #112

pmayd opened this issue Jul 28, 2023 · 2 comments
Labels

Comments

@pmayd
Copy link
Collaborator

pmayd commented Jul 28, 2023

We noticed that right now, when you use any of the note commands like @Karmabot note list the output is visible to anyone/others in the same channel (pybob used the command, I could make the screenshot):
image

This behavior is not intended, because it both is a privacy violation and increases the overall noise in the channels (something karmabot loves to do).

My proposed solution would be to move these messages into a private chat, so basically turn the output of these commands into DMs for the user.

@pmayd pmayd added the Bug label Jul 28, 2023
@pogross
Copy link
Member

pogross commented Aug 4, 2023

Easy fix would be to move the note command registration from CHANNEL_BOT_COMMANDS to DM_BOT_COMMANDS.

CHANNEL_BOT_COMMANDS = {
"add": add_command,
"age": pybites_age,
"joke": joke,
"note": note,
"tip": get_random_tip,
"topchannels": get_recommended_channels,
"zen": import_this,
}
DM_BOT_COMMANDS = {
"doc": doc_command,
"feed": get_pybites_last_entries,
"joke": joke,
"note": note,
"karma": get_karma,
"topchannels": get_recommended_channels,
"username": get_user_name,
"updateusername": update_username,
}

@bbelderbos
Copy link
Collaborator

I like that idea, this feature does not seem to have any business being used in channels :) - no?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants