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.

Sign up for GitHub

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

[ALL] Add validaton tags to all config models #57

Closed
gmbno opened this issue Aug 18, 2020 · 0 comments · Fixed by #77
Closed

[ALL] Add validaton tags to all config models #57

gmbno opened this issue Aug 18, 2020 · 0 comments · Fixed by #77

Comments

@gmbno
Copy link
Contributor

gmbno commented Aug 18, 2020

Ensure that configuration models implement validation tags (https://github.com/go-playground/validator)
@nabbar nabbar mentioned this issue Aug 23, 2020
Merged
nabbar pushed a commit that referenced this issue Aug 23, 2020
Merge pull request #58 from nabbar/issue_57
f3317d2 
Issue #57" add validator on ldap config" : 
- Add Validator from github.com/go-playground/validator for LDAP config
- Add error function to check if there are any parent error
- Bump dependancies
@nabbar nabbar linked a pull request Dec 31, 2020 that will close this issue
Refactor Packages #77
Merged
nabbar pushed a commit that referenced this issue Aug 16, 2023
Merge pull request #58 from nabbar/issue_57
5c62752 
Issue #57" add validator on ldap config" : 
- Add Validator from github.com/go-playground/validator for LDAP config
- Add error function to check if there are any parent error
- Bump dependancies
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Refactor Packages nabbar/golib
1 participant
@gmbno