Ajax headers for dashboard/config forms #3433

Merged
merged 5 commits into from Apr 29, 2016

Projects

None yet

3 participants

@pblasquez
Contributor

Please see #3113 for context, this is the dashboards & configs logical grouping.

pblasquez added some commits Apr 22, 2016
@pblasquez pblasquez Merge remote-tracking branch 'refs/remotes/librenms/master' 4510811
@pblasquez pblasquez Merge remote-tracking branch 'refs/remotes/librenms/master' add7aeb
@pblasquez pblasquez Ajax headers for dashboard/config forms
d2be07b
@laf laf commented on an outdated diff Apr 28, 2016
html/includes/forms/update-config-item.inc.php
@@ -11,6 +11,7 @@
* option) any later version. Please see LICENSE.txt at the top level of
* the source code distribution for details.
*/
+header('Content-type: application/json');
@laf
laf Apr 28, 2016 Member

Same issue here as the others :(

@laf
Member
laf commented Apr 28, 2016

Same as the last pr but only one file with an issue :(

pblasquez added some commits Apr 28, 2016
@pblasquez pblasquez Merge remote-tracking branch 'refs/remotes/librenms/master' into ajax…
…_headers_2
ee8268b
@pblasquez pblasquez Allow for separate headers upon conditional
1864269
@laf laf merged commit 1fc59a9 into librenms:master Apr 29, 2016

3 checks passed

Auto-Deploy Build finished. No test results found.
Details
Scrutinizer 4 new issues
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment