feature: Add osTicket Alert Transport #4791

Merged
merged 4 commits into from Oct 20, 2016

Projects

None yet

5 participants

@Rosiak
Contributor
Rosiak commented Oct 13, 2016

Please note

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

Add osTicket alert transport using the osTicket API.

@Rosiak
Contributor
Rosiak commented Oct 14, 2016

OSTicket text in sql schema needs to be changed to osTicket.
Will do that when I'm off work.

@Rosiak Rosiak update descr
ee4bd70
@laf
Member
laf commented Oct 14, 2016

I've got a PR outstanding which adds support for proxies to transports so this will need that adding in.

Aside from that, tested the webui side fine. No osTicket install to test further.

@laf
Member
laf commented Oct 14, 2016

Actually just realised my pr was updated so you'll need to make use of that now to add proxy support.

@laf
Member
laf commented Oct 19, 2016

bump @Rosiak

Rosiak added some commits Oct 20, 2016
@Rosiak Rosiak work work 1f3d9f3
@Rosiak Rosiak silly
6db07a2
@laf laf added Alerting and removed Conflict labels Oct 20, 2016
@scrutinizer-notifier

The inspection completed: 1969 Issues, 18 Patches

@f0o
f0o approved these changes Oct 20, 2016 View changes
@laf laf merged commit 23da8be into librenms:master Oct 20, 2016

2 checks passed

Auto-Deploy Build finished.
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@Rosiak Rosiak deleted the Rosiak:osticket-support branch Feb 5, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment