Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@MKRhere
Copy link
Member

@MKRhere MKRhere commented Apr 28, 2022

  • Upgrade to [email protected]; Bot API 6.0 support
  • Update minimist to latest, resolves a vulnerability
  • Added Markup.button.webApp helper
  • Experimental!: ⚠️ typegram & telegraf types exported as telegraf/types to be imported directly without relying on a separate dependency on typegram. The export interface is not stable. It may change at a later date

Try it out before it lands using npm install telegraf@canary

MKRhere added 2 commits April 29, 2022 02:39
Signed-off-by: Muthu Kumar <[email protected]>
@MKRhere MKRhere requested a review from KnorpelSenf April 28, 2022 21:12
MKRhere added 4 commits April 29, 2022 03:07
Reason: method could be called in `web_app_data`
Signed-off-by: Muthu Kumar <[email protected]>
Co-authored-by:  Eddy Otsutsuki  <@hypernova7>
Signed-off-by: Muthu Kumar <[email protected]>
Reasoning: web_app_data is only available for KeyboardButton Web Apps, which doesn't have web app query identifier and cannot use answerWebAppQuery

Use bot.tg.answerWebAppQuery instead

Signed-off-by: Muthu Kumar <[email protected]>
Copy link
Collaborator

@KnorpelSenf KnorpelSenf left a comment

Choose a reason for hiding this comment

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

LGTM, just a few nits here and there (optional to fix), plus a few questions about things I'm wondering about. Nothing that would be blocking.

Copy link
Collaborator

@KnorpelSenf KnorpelSenf left a comment

Choose a reason for hiding this comment

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

LGTM

MKRhere added 2 commits April 29, 2022 19:40
Signed-off-by: Muthu Kumar <[email protected]>
Signed-off-by: Muthu Kumar <[email protected]>
@MKRhere MKRhere merged commit 792e310 into v4 Apr 29, 2022
@MKRhere MKRhere deleted the feat-api-6-0 branch April 29, 2022 14:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants