-
-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
Pretty up Slack formatting. #15898
Pretty up Slack formatting. #15898
Conversation
* Normalize spaces by turning every instance of two or more spaces into one space. * Allow for custom URL links even when strip_tags() is in effect by transformaing "standard" markdown syntax into Slack link markdown after strip_tags() is run.
Perhaps a documentation update to mention the urls in markdown syntax would be good? |
While here, flesh out the Slack docs to match the configuration options one currently sees in the transport.
Document?! Don't we have people for that :-D . Good call. I added this bit about the links and fleshed out some other elements I thought were lacking. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM (but not tested as I don't use slack).
This pull request has been mentioned on LibreNMS Community. There might be relevant details there: |
DO NOT DELETE THE UNDERLYING TEXT
Please note
Testers
If you would like to test this pull request then please run:
./scripts/github-apply <pr_id>
, i.e./scripts/github-apply 5926
After you are done testing, you can remove the changes with
./scripts/github-remove
. If there are schema changes, you can ask on discord how to revert.