Skip to content

telegram_bot platform to only send messages (#8186) #2882

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

Merged
merged 3 commits into from
Jun 27, 2017
Merged

telegram_bot platform to only send messages (#8186) #2882

merged 3 commits into from
Jun 27, 2017

Conversation

azogue
Copy link
Member

@azogue azogue commented Jun 24, 2017

Description:

Doc for new telegram_bot/onlysend platform for send messages only.

Pull request in home-assistant (if applicable): home-assistant/core#8186

@mention-bot
Copy link

@azogue, thanks for your PR! By analyzing the history of the files in this pull request, we identified @sander76, @tchellomello and @fabaff to be potential reviewers.

Copy link
Contributor

@Landrash Landrash left a comment

Choose a reason for hiding this comment

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

Looks good and can be merged when parent PR is merged.

ha_release: 0.48
---

Telegram implementation to support **sending messages only**. Your Home Assistant does not have to be exposed to the Internet and there is no polling to receive messages sent to Bot.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggest a small rewrite.

Telegram implementation to support sending messages only. Your Home Assistant instance does not have to be exposed to the Internet and there is no polling to receive messages sent to the bot.

Copy link
Member Author

Choose a reason for hiding this comment

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

done! And changed the platform name as discussed in #8186

@Landrash Landrash added the new-integration This PR adds documentation for a new Home Assistant integration label Jun 26, 2017
@balloob balloob merged commit cfbea69 into home-assistant:next Jun 27, 2017
@azogue azogue deleted the telegram branch July 19, 2017 18:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new-integration This PR adds documentation for a new Home Assistant integration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants