RFR: 1660: Add confname field to slack log config [v2]

Erik Helin ehelin at openjdk.org
Mon Nov 7 09:20:43 UTC 2022


On Fri, 4 Nov 2022 13:51:17 GMT, Erik Joelsson <erikj at openjdk.org> wrote:

> > Hey Erik, nice work! Just two questions on the patch:
> > 
> > * why keep the field and config value `username`? If modern Slack hooks do not allow a username to be chosen, then shouldn't we remove that property from the config and the corresponding field from `BotSlackHandler`?
> 
> I'm not sure if this is a property of all slack webhooks or just the one app we are able to use at the moment. I figured we could leave the code in there for now and just not configure a username.

Ok, yeah, lets leave it in for a while.

-------------

PR: https://git.openjdk.org/skara/pull/1413


More information about the skara-dev mailing list