search

sum12 / python-rtmbot

My slack Bot
MIT License
0 stars 0 forks source link

Removing cron output with a special cron-status command #7

Open Diptanshu8 opened 7 years ago

Diptanshu8 commented 7 years ago

Presently, we are having a cronned output to a seperate channel in slack. This is invariantly giving rise to distinct channels for every bot cron output in the same slack group. So a better approach would be to append the last status of cron executed by bot for every job to the cron handler class so that the status can be extracted by direct communication from the bot rather than grepping the output of the seperate cron channel.

sum12 commented 7 years ago

I will be happy to have such a functionality.

sum12 commented 7 years ago

Or may be how is the "inform of replying to the owner".

On 26-Apr-2017 10:41, "Diptanshu Jamgade" notifications@github.com wrote:

Presently, we are having a cronned output to a seperate channel in slack. This is invariantly giving rise to distinct channels for every bot cron output in the same slack group. So a better approach would be to append the last status of cron executed by bot for every job to the cron handler class so that the status can be extracted by direct communication from the bot rather than grepping the output of the seperate cron channel.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/sum12/python-rtmbot/issues/7, or mute the thread https://github.com/notifications/unsubscribe-auth/ADIhEyta62Z659D_eKa0VfgFFB12M0Xtks5rzwMkgaJpZM4NIioK .

Diptanshu8 commented 7 years ago

Do you suggest to add this "inform to the owner" feature in the .conf file? Then in that case, all the cronned outputs would be generated repeatedly. But cron-status would work only for the last executed cron info. No need to get a .conf modification. Morever, if we use "inform to the owner" approach, do you mean to say that we use the config or some other approach?