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

Copy over some gag commands from old zodbot #22

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

gotmax23
Copy link
Contributor

No description provided.

Copy link
Collaborator

@ryanlerch ryanlerch left a comment

Choose a reason for hiding this comment

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

IMHO, i'm happy leaving these joke commands in old zodbot.

However, if we continue with them here, Just wondering if this would be worth using a passive handler in kind of the same way as we do for the pagureio aliases. I suggest this for two reasons:

  1. we could then perhaps put these joke commands in configuration, since they are basically the same thing, responding with an emoji and a message.
  2. they stay out of the main !help -- which i have been working to try to group things under subcommands a lot more to stop it just being a giant list.

Also, with the emoji reactions -- currently it emoji-responds to the ! command message, then replies with an additional message from zodbot. is that what you were intending on happening?

@gotmax23
Copy link
Contributor Author

Thanks for the feedback.

IMHO, i'm happy leaving these joke commands in old zodbot.

Fair enough. If the consensus is to get rid of them (/cc @nirik, @abompard in case you have thoughts), I won't be hurt by it.

we could then perhaps put these joke commands in configuration, since they are basically the same thing, responding with an emoji and a message.

I could look into that.

they stay out of the main !help -- which i have been working to try to group things under subcommands a lot more to stop it just being a giant list.

I edited the help code to exclude commands from the help if they don't have help text and then didn't add help text here, so they don't show up in !help at all.

Also, with the emoji reactions -- currently it emoji-responds to the ! command message, then replies with an additional message from zodbot. is that what you were intending on happening?

Yes, that was the intention.

@nirik
Copy link
Member

nirik commented Oct 24, 2023

I think keeping them would be kinda nice and preserve a bit of history, but if everyone else wants them gone... ok.

I suppose if we wanted to be elaborate we could add a way for these to just be configuration/configurable, ie,

!addgag name reationemoji reply subject

so

!addgag fires 🔥 adamw fires {{ subject }}

and then

!fires nirik
adamw fires nirik
🔥

But thats getting pretty elaborate. ;)

Copy link

@Conan-Kudo Conan-Kudo left a comment

Choose a reason for hiding this comment

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

I like seeing this come over to the new zodbot. It's part of our chat culture. That said, one small remark.

async def fire(self, evt: MessageEvent, firee: str) -> None:
await evt.mark_read()
await evt.react("🔥")
await evt.reply(f"adamw fires {firee}")

Choose a reason for hiding this comment

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

This should be @adamwill:fedora.im

Copy link
Member

Choose a reason for hiding this comment

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

What does Adam think about being pinged each time someone uses this command?

Choose a reason for hiding this comment

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

Well, it's @AdamWill's thing, we can ask. Though, he was pinged before when it was used.

Choose a reason for hiding this comment

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

i think i'd rather not be pinged. thanks!

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.

None yet

6 participants