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

Allow underscore override per RR #146

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

mzagrabe
Copy link
Contributor

I'd like to add one or two RR's to a domain, but I don't want to toggle the config option:

ALLOW_UNDERSCORES_IN_DEVICE_NAMES

Instead, I've added a new config option:

UNDERSCORES_IN_DEVICE_NAMES_OVERRIDE_ALLOWED

that if set will display a checkbox on host.html when adding a record from management/zone_tasks.html. Checking the checkbox will allow that record to contain underscores.

Convert all tabs to 4 spaces for subroutine _validate_args.
Remove trailing whitespace and canonicalize things according to Perl
Best Practices. This is in anticipation of further changes coming.
Setting this to true will give the web user an underscore "override" checkbox
when creating a record within a domain.
Use extra_argv to pass UI checkbox value to this function to determine
if underscore validation needs to occur.
Pass "allow_underscore_override" value into _validate_args.
If configuration values are set appropriately then return a string and
checkbox for overriding underscore validation.
Potentially alter form with underscore validation override checkbox.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant