Skip to content

More data validation fixes/additions#102

Merged
lsipii merged 3 commits intomainfrom
VFD-339-fix-validations
Nov 24, 2023
Merged

More data validation fixes/additions#102
lsipii merged 3 commits intomainfrom
VFD-339-fix-validations

Conversation

@lsipii
Copy link
Copy Markdown
Contributor

@lsipii lsipii commented Nov 22, 2023 •
edited
Loading

  • validates the productizer residency and language fields against the ISO-codes
    • just like is being validated in the user-controller side
  • allow to save person basic information fields as null as to clear the field.

@lsipii
feat: apply the language and residency codesets-based validations to …
16a21b7 
…the productizer routes, allow nullable basic infos
@lsipii lsipii requested review from LauriGofore and removed request for LauriGofore November 22, 2023 09:55
@lsipii lsipii marked this pull request as draft November 22, 2023 09:58
@lsipii lsipii requested a review from LauriGofore November 22, 2023 10:07
@lsipii lsipii marked this pull request as ready for review November 22, 2023 10:07
LauriGofore
LauriGofore approved these changes Nov 22, 2023
View reviewed changes
Copy link
Copy Markdown
Contributor

@LauriGofore LauriGofore left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@lsipii lsipii merged commit a822ace into main Nov 24, 2023
@lsipii lsipii deleted the VFD-339-fix-validations branch November 24, 2023 08:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@LauriGofore LauriGofore LauriGofore approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@lsipii @LauriGofore