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

add quick_reply_options #1364

Closed
wants to merge 1 commit into from
Closed

add quick_reply_options #1364

wants to merge 1 commit into from

Conversation

om8007
Copy link

@om8007 om8007 commented May 15, 2020

add quick_reply_options for passing options object for quick_reply_type = 'options'

adding 'options' in message_data['quick_reply'] allows sender to pass options object for quick_reply_type = 'options'. Without 'options' it raises tweepy.error.TweepError: [{'code': 214, 'message': 'event.message_create.message_data.quick_reply: quick_reply per-type node not found for quick_reply type options'}]

adding 'options' in message_data['quick_reply'] allows sender to pass options object for quick_reply_type = 'options'. Without 'options' it raises tweepy.error.TweepError: [{'code': 214, 'message': 'event.message_create.message_data.quick_reply: quick_reply per-type node not found for quick_reply type options'}]
Copy link
Member

@Harmon758 Harmon758 left a comment

Choose a reason for hiding this comment

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

Location and text input quick replies should be supported as well.

@Harmon758 Harmon758 added Feature This is regarding a new feature Improvement This is regarding an improvement to an existing feature Need Follow-Up This needs to be followed up on to be actionable labels Dec 26, 2020
@Harmon758
Copy link
Member

Sorry, due to Location and Text Input Quick Replies still being documented, I didn't realize that they had been retired from the API.

However, since Options Quick Replies are the only type now, quick_reply_options can simply replace quick_reply_type as a parameter, with the type set to "options" if options are passed, and with this PR being stale, I've gone ahead and done that with d8b02de.

@Harmon758 Harmon758 closed this Jan 11, 2021
@Harmon758 Harmon758 added Stale This is inactive, outdated, too old, or no longer applicable and removed Need Follow-Up This needs to be followed up on to be actionable labels Jan 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature This is regarding a new feature Improvement This is regarding an improvement to an existing feature Stale This is inactive, outdated, too old, or no longer applicable
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants