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.

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

Add minimum password length setting #10867

Merged
merged 1 commit into from Nov 25, 2019
Merged

Conversation

murrant
Copy link
Member

@murrant murrant commented Nov 22, 2019

An example of how to add a setting to the Web UI.

If you can add text for other languages you should.

To update the mix-manifest.json file, run ./lnms tranlation:generate

DO NOT DELETE THIS TEXT

Please note

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

Testers

If you would like to test this pull request then please run: ./scripts/github-apply <pr_id>, i.e ./scripts/github-apply 5926
After you are done testing, you can remove the changes with ./scripts/github-remove. If there are schema changes, you can ask on discord how to revert.

@murrant murrant added the WebUI label Nov 22, 2019
@murrant murrant added this to the 1.58 milestone Nov 25, 2019
@murrant murrant merged commit 04b65d3 into librenms:master Nov 25, 2019
@murrant murrant deleted the add-setting branch November 25, 2019 04:11
@murrant
Copy link
Member Author

murrant commented Nov 25, 2019

This pull request has been mentioned on LibreNMS Community. There might be relevant details there:

https://community.librenms.org/t/v1-58-release-changelog-november-2019/10224/1

@murrant
Copy link
Member Author

murrant commented Nov 25, 2019

This pull request has been mentioned on LibreNMS Community. There might be relevant details there:

https://community.librenms.org/t/v1-58-november-2019-info/10225/1

@lock lock bot locked as resolved and limited conversation to collaborators Jan 24, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant