Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR adds the possibility to send the new quick reply types
location
,user_email
anduser_phone_number
. (More info here)This is how we could send
QuickReplies
before: (and is still possible)Now you get the same result with the new
QuickReplyButton
class:And additionally now for the new
QuickReply types
:What I am not satisfied with is:
Maybe it would be nicer to create own method for
createLocation
,createUserEmail
andcreateUserPhoneNumber
but I tried to make it consistent with other templates andcreate
normally takes the title.The
convertQuestion
method from the FacebookDriver now looks a little bit messy because it needs to support both button types. Maybe it would be nicer to use aQuickReply
class instead of theQuestion
one. But maybe it is ok so for now.