-
Notifications
You must be signed in to change notification settings - Fork 808
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Create a publisher for seq #24
Conversation
Hi @Franklin89 Let me this week to check this PR, thanks! |
@Franklin89 could you address the merge problem in this PR? Thanks |
@CarlosLanderas of course, do you have an idea for the message that should be sent to SEQ? Currently it just shows this, but I have no Idea at the moment what the best message is for a User: |
src/HealthChecks.Publisher.Seq/DependencyInjection/SeqHealthCheckBuilderExtensions.cs
Outdated
Show resolved
Hide resolved
Hi @Franklin89 can you fix this minor issues? |
@unaizorrilla I made the requested changes. Let me know if there is anything else that should be done. |
Merged with some minor changes! Matteo, can you try to add a Seq docker image in the docker-compose in order to test this publisher? |
Looking for suggestions and feedback on this. Also for the message template, I am unsure about a good message that could be displayed.
Usage:
Dashboards similar to the ones described for App Insights can be created: