-
Notifications
You must be signed in to change notification settings - Fork 3
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
✨ [#132] added api validation to check if nummeraanduiding_i… #133
Conversation
cc4d645
to
e46ee53
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@bart-maykin waar komt dit issue vandaan?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Marking as request changes to make it clear to something is up :)
In het gegevenswoordenboek is het ID veld gemarkeerd als required. En omdat dit in een gegevens groep zit, is het logisch dat dit alleen het geval is als de gegevens groep ingevuld is. Maar de functionaliteit miste we nog dus daarom had ik deze issue aangemaakt. |
e7f2236
to
a27774c
Compare
…n when gegevens groep is filled in
a27774c
to
adfd430
Compare
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## master #133 +/- ##
==========================================
+ Coverage 96.67% 96.88% +0.21%
==========================================
Files 180 178 -2
Lines 7364 7416 +52
==========================================
+ Hits 7119 7185 +66
+ Misses 245 231 -14 ☔ View full report in Codecov by Sentry. |
Zie ticket waarom gesloten. |
…d is given when gegevens groep is filled in
fixes #132
added validation to gegevengroepen to see if nummeraanduiding_id must be required or not.