Comment 4 for bug 1879517

Revision history for this message
Jane Sandberg (sandbergja) wrote :

Thank you for your code, Mike and Kyle! It works well, but I have a concern that the isDataValid structure is essentially duplicating the existing validation features in Angular.

I've been playing with an alternative patch that uses the existing validation features: https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/sandbergja/lp1879517_alternative_patch . It's not quite there (I tried to set a default end date in the future, but couldn't figure out how...), but hopefully it at least demonstrates the alternative approach.