fix: Added support for https links in alerts procedure url #4872

Merged
merged 1 commit into from Nov 1, 2016

Projects

None yet

5 participants

@laf
Member
laf commented Oct 24, 2016

Please note

Please read this information carefully. You can run ./scripts/pre-commit.php to check your code before submitting.

Fixes: #4865

also took the chance to shrink the procedure open button as well.

image

@laf laf fix: Added support for https links in alerts procedure url
cb48ae1
@scrutinizer-notifier

The inspection completed: No new issues

@sliddjur

Is it possible to add support for %placeholder in links while you're at it? :)))

@laf
Member
laf commented Nov 1, 2016

bump

@f0o
f0o approved these changes Nov 1, 2016 View changes
@f0o f0o merged commit 4a854e0 into librenms:master Nov 1, 2016

2 checks passed

Auto-Deploy Build finished.
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@laf laf deleted the laf:issue-4865 branch Nov 1, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment