Skip to content
This repository has been archived by the owner on Aug 26, 2022. It is now read-only.

meta: convert validation to ozzo #227

Open
adamdecaf opened this issue Oct 6, 2020 · 1 comment
Open

meta: convert validation to ozzo #227

adamdecaf opened this issue Oct 6, 2020 · 1 comment
Assignees
Labels
bug Something isn't working enhancement New feature or request

Comments

@adamdecaf
Copy link
Member

Customers Version: v0.5.0-dev

What were you trying to do?
All data objects need to be properly validated. This involves ensuring their formatting is accepted for the underlying type or they're containing only acceptable characters.

Library: github.com/go-ozzo/ozzo-validation
Example: https://github.com/moov-io/identity/blob/master/pkg/client/validation.go

What did you expect to see?
Data should only be accepted if it is valid.

Implementation Notes
Please do this one package/file at a time. A big change across the project might conflict with other changes in-flight.

@adamdecaf adamdecaf added bug Something isn't working enhancement New feature or request labels Oct 6, 2020
@adamdecaf adamdecaf added this to To do in Current Work Oct 6, 2020
@adamdecaf adamdecaf added this to the v0.6 Release milestone Oct 9, 2020
@vxio vxio self-assigned this Oct 11, 2020
@vxio
Copy link
Member

vxio commented Oct 23, 2020

We should also add a normalization layer before performing the validations.

@vxio vxio moved this from To do to In progress in Current Work Nov 3, 2020
@adamdecaf adamdecaf removed this from the v0.6 Release milestone Feb 13, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working enhancement New feature or request
Projects
No open projects
Current Work
  
In progress
Development

No branches or pull requests

2 participants