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

Jump to bottom

Mark visually the mandatory configuration settings #15

Closed
oanat opened this issue Aug 22, 2019 · 0 comments
Closed

Mark visually the mandatory configuration settings #15

oanat opened this issue Aug 22, 2019 · 0 comments
Assignees
@acotiuga
Labels
Priority: Minor Type: Improvement
Milestone
1.7

Comments

@oanat
Copy link

@oanat oanat commented Aug 22, 2019

The 5 mandatory configuration settings (the four connection settings and the ACTIVE DIRECTORY BASE DN) should be marked as mandatory to help the user easily connect his AD server.
@acotiuga acotiuga added this to the 1.7 milestone Aug 31, 2019
acotiuga added a commit that referenced this issue Sep 10, 2019
@acotiuga
Mark visually the mandatory configuration settings #15
d79a161 
* use the HTML required attribute as a mean of form validation, in order to prevent the save with empty values
@acotiuga acotiuga self-assigned this Sep 10, 2019
@acotiuga acotiuga closed this Sep 10, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@acotiuga acotiuga

Labels
Priority: Minor Type: Improvement
Projects
None yet
Milestone
1.7
Development

No branches or pull requests
2 participants
@oanat @acotiuga