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

[API v4.0] Publish new methods and message types #625

Merged
merged 5 commits into from
Oct 12, 2018

Conversation

kamikazechaser
Copy link
Collaborator

@kamikazechaser kamikazechaser commented Aug 1, 2018

  • All tests pass
  • I have run npm run doc

Description

Update the library with minor changes from the latest bot API update (v4.0 released on 26/07/2018).

Add new methods:

  • editMessageMedia
  • sendAnimation

Add new message types:

  • passport_data
  • animation

References

@TorlanDelta
Copy link

So how long till this gets merged? What are we waiting for?

@nathenapse
Copy link

please merge this

@kamikazechaser
Copy link
Collaborator Author

@yagop @GochoMugo If possible, merge this asap.

@yagop yagop merged commit 42b31fb into yagop:master Oct 12, 2018
@yagop
Copy link
Owner

yagop commented Oct 12, 2018

Merged. Wold be good to have tests too.

@kamikazechaser
Copy link
Collaborator Author

I'll add the tests asap.

kamikazechaser added a commit to kamikazechaser/node-telegram-bot-api that referenced this pull request Oct 20, 2018
- Update badge to reflect the current update (yagop#625)
- Fix broken BitHound badge
@kamikazechaser kamikazechaser mentioned this pull request Oct 20, 2018
kamikazechaser added a commit to kamikazechaser/node-telegram-bot-api that referenced this pull request Oct 26, 2018
 Follow up update to yagop#625

 - Tests for both methods (sendAnimation, editMediaMessage) have been
 added. editMediaMessage is nested under sendAnimation, hence both tests
 are technically under a single block.

- Add an improvement/minor fix to the editMediaMessage method, where we
now stringify the first parameter (media) object interally within the
library. This allows the lib user to simply pass an object as the 1st
param without stringifying it.
kamikazechaser added a commit to kamikazechaser/node-telegram-bot-api that referenced this pull request Oct 26, 2018
 Follow up update to yagop#625

 - Tests for both methods (sendAnimation, editMediaMessage) have been
 added. editMediaMessage is nested under sendAnimation, hence both tests
 are technically under a single block.

- Add an improvement/minor fix to the editMediaMessage method, where we
now stringify the first parameter (media) object interally within the
library. This allows the lib user to simply pass an object as the 1st
param without stringifying it.
kamikazechaser added a commit to kamikazechaser/node-telegram-bot-api that referenced this pull request Oct 26, 2018
- Update badge to reflect the current update (yagop#625)
- Fix broken BitHound badge
kamikazechaser added a commit that referenced this pull request Nov 2, 2018
* Update README badges

- Update badge to reflect the current update (#625)
- Fix broken BitHound badge

* Update badges

- Replace coveralls with codecov
- Add new appveyor badge

* Fix minor typo
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.

4 participants