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

@azogue can you have a look and confirm this? 😈

Perfect, I passed by a few minutes ago and saw that it was clearly improveable, but laziness is very bad ...

I think if you set the DEBUG logging level for the telegram_bot component, you can get the chat_id's without leaving HA, but I'm not sure.

I've checked it, when you add your bot to a new chat group, it will raise an error in the Home Assistant logs like that:

2017-07-30 17:08:53 ERROR (MainThread) [homeassistant.components.telegram_bot]
Incoming message is not allowed ({'new_chat_participant':
{'first_name': 'MyBot', 'id': 123456789, 'username': 'my_bot_name'}, 'new_chat_members': [{'first_name': 'MyBot', 'id': 123456789, 'username': ''my_bot_name'}], 'chat': {'id': -246188003, 'title': 'Chat group title', 'type': 'group', 'all_members_are_administrators': True}, 'message_id': 2740, 'date': 1501427333, 'new_chat_member': {'first_name': 'MyBot', 'id': 123456789, 'username': ''my_bot_name'}, 'from': {'last_name': 'Your last name', 'language_code': 'es', 'first_name': 'Your name', 'id': 987654321, 'username': 'your_username'}})

Because the incoming message is logged, you can get the group chat_id from incoming_msg['chat']['id'].

Hi @dullage,

Unless I list both the chat id and my user id in the allowed_chat_ids parameter then I receive a "Incoming message is not allowed" message in the HA log.

When targeting groups, you need to include both the group id (the negative number) and all the participants ids, because for listening the messages sent by them, all ids need to be allowed previously.

If a new user tries to send a message, HA will raise another error with all needed data to get the new id, so no need to leave HA.

@azogue Trying to follow the conversation. Rewrite this and merge or close since it's close enough?

Hi All,

My main goal with the PR was to clarify that we need to add user ids as well as chat ids if we are using a group. Azogues findings on being able to use the HA log to obtain these ids is very useful and I've added another commit which hopefully addresses this.

@Landrash, looks good to me!

Thank you @dullage & @azogue 🍰