This repository has been archived by the owner on Apr 12, 2019. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 1
Validate Reference #155
Comments
Such a check should be handled by the adapter itself. #161 |
Well you're right that the adapter should check the length itself, but I'm not sure if it is mandatory. According to #2 it is not really clear to me. |
I'd suggest #2 has to be changed, and that the Pakt-adapter is responsible for anything that goes beyond the standard node validation. I.e. if the field is indeed mandatory for the Pakt-adapter, it'll have to validate it itself. |
After discussing with @hoehnb this issue will be moved back to the backlog as it would need further discussions. For the moment it is fine if the reference is optional. |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add Validation for reference / or internal referenz as it is called in the code.
It is mandatory. - I'd recommend to not check the length. @Plattnea suggested a max length of 35 characters which is very PAKT specific.
The text was updated successfully, but these errors were encountered: