-
Notifications
You must be signed in to change notification settings - Fork 5k
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
CAMEL-12478: Allow use of custom keyboard #2318
CAMEL-12478: Allow use of custom keyboard #2318
Conversation
I separated in 5 commits, but when the review is done I should probably squash them up in one, right? |
@@ -0,0 +1,53 @@ | |||
package org.apache.camel.component.telegram.model; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add the License header here.
@@ -0,0 +1,121 @@ | |||
package org.apache.camel.component.telegram.model; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
License header please.
9bbb2e5
to
7cf6ca9
Compare
|
||
[source,java] | ||
--------------------------------------------------------- | ||
from("telegram:bots/123456789:AAE_dLq5C19xwGjw3yiC2NvEUrZcejK21-Q987654321:AAE_dLq5C19xwOmg5yiC2NvSrkT3wj5Q1-L") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please make sure you don't use a private bots, as the access key may be abused.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I used the same access key available in the documentation. See other examples on it. This key is not mine.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So it should be a random one I've generated (I hope 😄)
LGTM now, @WillemJiang would you like to merge? |
Thanks, this has been merged on master. |
Entesb 22164 713
As described here, it will permit the usage of custom keyboard in order to increase the interaction with the user as it's a lot easier to select an option instead of ask the user writing one.