Add Telegram API #72
Add Telegram API #72
Conversation
Signed-off-by: Real96 <[email protected]>
Scripts/1.ahk
Outdated
| global telegramToken := "" ; write the telegram bot token value inside the quotes | ||
| global telegramChatID := "" ; write the telegram chat id value inside the quotes | ||
| global telegramTopicID := "" ; write the telegram topic id value inside the quotes or leave blank |
There was a problem hiding this comment.
You would need to set up the variables in the Settings.ini file, to call them after the other globals.
You'll have to do the GUI as for discord id and discord webhook for telegramChatId and telegramTopicId
There was a problem hiding this comment.
You would need to set up the variables in the Settings.ini file, to call them after the other globals. You'll have to do the GUI as for discord id and discord webhook for telegramChatId and telegramTopicId
If you read the issue I linked, @Arturo-1212 doesn't want to add the feature in the GUI
There was a problem hiding this comment.
ok for the GUI but vars are better set in Settings.ini
There was a problem hiding this comment.
ok for the GUI but vars are better set in
Settings.ini
But the Settings.ini file doesn't exist when you start the bot for the first time. This way you should start the bot first and then edit the settings. Isn't it?
There was a problem hiding this comment.
To be tested: if you commit a Settings.ini with these vars and you start the GUI and enter the fields, it will merge the Telegram vars and the ones added by the GUI.
If this solution doesn't work, there's really no other way than to declare them in the code.
There was a problem hiding this comment.
Seems like it worked, so i made a new commit. The only downside i see is that the Settings.ini file must be removed from the .gitignore file, so when the file is edited it will appear as a new file to be pushed from the local cloned repo of the project
There was a problem hiding this comment.
Okey good ^^. Now we wait the approve of @Arturo-1212.
Signed-off-by: Real96 <[email protected]>
|
Like a mentioned in that post this won't be added to the main branch at this time. |
Implement what was asked in #68