JADNC: Required Input validation disabled for partial patching / relationships

[sasman0001]

Fixes #472

This feature allows the Required validator to be disabled. This will allow partial patching in the case that the attribute is excluded from a patch. Validation is applied when the attribute is present in post or patch.

Required validation also disabled for a relationships attributes (in attempt to fix #472 (comment)).

[sasman0001]

FYI. Integration / unit tests succeed locally, but for some reason are failing according to the "AppVeyor build." Will investigate.

[bart-degreed]

This looks great! I agree this new approach is an improvement over what you submitted before. Thanks.

[bart-degreed]

The cibuild failure is likely unrelated to your work. Some tests are still not properly isolated, which makes them fail occasionally (tracked at #715). Usually pushing another empty commit makes the build succeed.

[bart-degreed]

Please merge the latest master into your branch and see if all still works before we take this PR.

[sasman0001]

Bart, I pulled the latest master, but then thought since this is my fork, it may not actually be pulling the JADNC master. I am not familiar with "forking" and am unsure how to merge JADNC master into my fork.

I have made the other changes you requested. Thank you for all the great feedback.

[bart-degreed]

git fetch upstream git checkout yourbranch git merge upstream/master In the same way you can sync your forked master branch. Hope this helps.

[bart-degreed]

This can inherit from the non-generic Identifiable, which is easier to understand for newcomers (see my example). Also this is a nice opportunity to show usage of the AllowEmptyStrings parameter so users will know it exists. Can you use my example from earlier comments instead?

Never mind, I'll change this.

[bart-degreed]

Thanks a lot, Sarah!