Skip to content
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.

Sign up for GitHub

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

Jump to bottom

change gearman forwarder to use twig-based templates #310

Closed
toschneck opened this issue Feb 19, 2018 · 0 comments
Closed

change gearman forwarder to use twig-based templates #310

toschneck opened this issue Feb 19, 2018 · 0 comments
Assignees
@joroboro
Labels
area:forwarder:gearman Prio: Medium Type: Feature
Milestone
rel-v1.2

Comments

@toschneck
Copy link
Contributor

Use Twig templates similar to check_mk implementation #176
This was referenced Feb 19, 2018
Open
Closed
@toschneck toschneck added this to the rel-v1.2 milestone Feb 19, 2018
@joroboro joroboro mentioned this issue Feb 19, 2018
Open
@joroboro joroboro changed the title change gearman forwarder to use twig-based templates change gearmacahngen forwarder to use twig-based templates Feb 19, 2018
@ghost ghost added IN PROGRESS and removed READY labels Feb 19, 2018
@toschneck toschneck changed the title change gearmacahngen forwarder to use twig-based templates change gearman forwarder to use twig-based templates Feb 20, 2018
joroboro pushed a commit that referenced this issue Feb 26, 2018
#310 New JTwigFunction introduced. Existing JTwig functions refactored.
ce0868f
joroboro pushed a commit that referenced this issue Feb 26, 2018
#310 Twig templates for Gearman introduced.
89b8b37
joroboro pushed a commit that referenced this issue Feb 26, 2018
#310 Use Twig templates for Gearman
8f9b620
joroboro pushed a commit that referenced this issue Feb 26, 2018
#310 Tests for GearmanTemplateOutputBuilder added and refactoring done.
779d05f 
 - some old Nagios- and NagiosBuilder- Objects have been removed
 - some objects from gearman package moved to the forward package, since they were not only used for gearman, but also for other forwarder too.
@joroboro joroboro mentioned this issue Feb 27, 2018
Merged
@ghost ghost added TO REVIEW and removed IN PROGRESS labels Feb 27, 2018
toschneck pushed a commit that referenced this issue Mar 7, 2018
review #310: do some smaller refactoring, add nullchecks and some tests
4c8364d
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@joroboro joroboro

Labels
area:forwarder:gearman Prio: Medium Type: Feature
Projects
None yet
Milestone
rel-v1.2
Development

No branches or pull requests
2 participants
@toschneck @joroboro