-
-
Notifications
You must be signed in to change notification settings - Fork 103
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
get_message for messages of type interactive or list_reply #68
Comments
Hi @soerenetler Didn't quite get you on the second part, Can you please clarify more on that ? |
If the user does not write the answer but uses buttons the answer ist of type |
@Kalebu I'm actually trying some fixes locally for this. Is it ok to create a Message() object which is returned to the user on get_message() method call? |
I like this approach :) I think this also goes along with #32 |
I've create a fork of this repo! |
@soerenetler hello! |
Hello again,
I think get_message should be renamed to get_message_text (current naming is ambiguous, it is not clear if the function returns the message object or the text). And I would argue it should return the
button_reply -> title
for messages of typeinteractive
and thelist_reply -> title
andlist_reply -> description
forlist_reply
What do you think?
The text was updated successfully, but these errors were encountered: