Closed alanedwardes closed 7 years ago
Will take a look at this tomorrow, though quick question: is it compatible with previous version ? Thanks 👍
Thanks. It is yeah, existing overload is still there, it calls the new extension method which accepts the options object.
All looks good for me - please just update README.md with some example.
I merged it and added example.
This ended up being more of a refactor than I planned, but I made the following changes:
ShowDefaultAttachments
,ShowExceptionAttachments
andAttachmentColors
in a newSlackSinkOptions
objectSeqLoggerConfigurationExtensions
toSlackLoggerConfigurationExtensions
dynamic
from JSON serialisationDispose()
on the batch sink, soHttpClient
can be disposedThe attachments toggling was the main thing I wanted to add, the messages were too big on Slack - I could fit 5 log messages on my screen at once.