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

Posting to another channel #23

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

sorrowless
Copy link
Contributor

Bot will have an option to post message in channel different than original
one it was listened to. To use this ability, you should add
'target_channel' with channel name as an argument of your 'listen_for'
decorator.

Stanislaw Bogatkin added 2 commits January 4, 2016 18:12
Change signature of 'listen_for' method by deleting **options
signature from it, as it was broken anyway - you had had no
option to run this with additional named options, but it was
needed to have an ability of pushing options of decorated
function. Instead of it some 'academic' code was added - now
'listen_for' method is a decorator factory which returns
decorator that have new function declared inside. It allows to
push named options of decorated function to it and get rid of
wrong signature of factory.
Bot will have an option to post message in channel different than original
one it was listened to. To use this ability, you should add
'target_channel' with channel name as an argument of your 'listen_for'
decorator.
@nficano
Copy link
Owner

nficano commented Jan 11, 2016

@sorrowless what's the use case for this?

@sorrowless
Copy link
Contributor Author

@nficano for creating bot only channels, of course :)

@punchagan
Copy link

This seems like a reasonable feature to have. For instance, you could have a channel where all the links posted on different channels could be aggregated.

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

Successfully merging this pull request may close these issues.

3 participants