Skip to content

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

Merged
balloob merged 3 commits into
home-assistant:nextfrom
azogue:telegram
Jun 27, 2017
Merged

telegram_bot platform to only send messages (#8186)#2882
balloob merged 3 commits into
home-assistant:nextfrom
azogue:telegram

Conversation

@azogue
Copy link
Copy Markdown
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
Copy Markdown

@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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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