JetBrains / teamcity-slack-notifier

TeamCity Slack Notifier plugin
https://jetbrains.com/teamcity
Apache License 2.0
7 stars 8 forks source link

Allow updating messages instead of posting new ones #3

Open sugdyzhekov opened 1 year ago

sugdyzhekov commented 1 year ago

Slack API allows updating existing messages. For most cases, you need only a fact of the current state: "Started," "Finished," and "Failed." Slack history can be cleaner if the plugin updates the message in the scope of one build run.

Two messages are handy to estimate how long the build was run. So, the requested behavior should be optional or provide time in the update to handle this case.

jonnyzzz commented 1 year ago

Actually it may even maintain the history in the thread, while updating the current status in the first/head message.

On Sun 2. Oct 2022 at 15:21, Sergei Ugdyzhekov @.***> wrote:

Slack API allows updating existing messages. For most cases, you need only a fact of the current state: "Started," "Finished," and "Failed." Slack history can be cleaner if the plugin updates the message in the scope of one build run.

Two messages are handy to estimate how long the build was run. So, the requested behavior should be optional or provide time in the update to handle this case.

— Reply to this email directly, view it on GitHub https://github.com/JetBrains/teamcity-slack-notifier/issues/3, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAB6TL26EX5LOZIQOVAM7ZTWBGD5RANCNFSM6AAAAAAQ2435RY . You are receiving this because you are subscribed to this thread.Message ID: @.***>

-- Best regards, Eugene Petrenko https://jonnyzzz.com