Make Global Search Limit Configurable #2957

Merged
merged 4 commits into from Feb 20, 2016

Projects

None yet

4 participants

@Rosiak
Contributor
Rosiak commented Feb 8, 2016
@Rosiak Rosiak Make Global Search Limit Configurable
f4d5e21
@laf
Member
laf commented Feb 9, 2016

It would be good to make this a WebUI config option. You can use html/pages/settings/external.inc.php as an example of doing something like html/pages/settings/webui.inc.php

@laf
Member
laf commented Feb 11, 2016

Commit to add db config support you can use (forgot to remove includes/defaults.inc.php option so ensure you do that).

9fcb93b

@Rosiak Rosiak Move function to WebUI
Based on Neil’s suggestion.
1c630b0
@Rosiak
Contributor
Rosiak commented Feb 13, 2016

Thanks Neil.
Suggested changes has been merged in.

!!!! Notice the SQL schema, DO NOT MERGE until we are at that point !!!!

@Rosiak Rosiak Remove space
fd3c7f4
@f0o
Member
f0o commented Feb 17, 2016

Propose moving the schema down to 102 for immediate merging

@laf
Member
laf commented Feb 20, 2016

@Rosiak move to 102.sql? We can merge if so.

@Rosiak
Contributor
Rosiak commented Feb 20, 2016

Moved.

@laf laf removed the Blocker label Feb 20, 2016
@laf laf merged commit 46a17ea into librenms:master Feb 20, 2016

1 check failed

Auto-Deploy Build finished. No test results found.
Details
@Rosiak Rosiak deleted the Rosiak:issue-2557 branch Feb 5, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment