archive
Include optional params on update?
Hello
Thanks for this action, very nice. One thing I'm wondering is whether it would be possible to add support for including optional params on message update. I'm thinking of appending block sections as new details become available during a build.
Also, would it be possible to include an example showing the use of attachments/blocks? I managed it but found it a bit tricky, and it wasn't 100% clear on what other params were mandatory - I had to go to the code for that. Also, I believe it's possible to send just a block section, so maybe don't make the text field mandatory?
Thanks again.
Hi!
It's a good idea. Need to check what it would mean for the implementation.
From what I know, today blocks don't support "append" in the API, you have to push the old + the new to fake that behavior.
Sorry for the long delay. To enable append update we need to change the permission level to allow read. Since I need to read the old post, append, and then update. From I see, the slack api is only replace, not append.
Would that feel ok, to give some more permissions?
What is the feeling of this? This capability would require the action to have vastly more permissions, which I don't really like.
@halberom - Support for blocks is now released https://github.com/archive/github-actions-slack/releases/tag/v2.6.0.
It will not support append mode, it will overwrite what was before.