search

remoteoss / json-schema-form

https://json-schema-form.vercel.app
MIT License
70 stars 7 forks source link

[JSON Logic] Part 5: Add general error handling #39

Closed brennj closed 10 months ago

brennj commented 10 months ago

When evaluating a rule to be validated, thats when we can get insight whether to throw or not based on the lack of existence of a field.

Please see references of this in the example schemas provided in this PR.

brennj commented 10 months ago

idle thought: these merges from downstream changes are breaking my heart

brennj commented 10 months ago

This one is properly ready for another review.

johnstonbl01 commented 10 months ago

idle thought: these merges from downstream changes are breaking my heart

Yeah... This type of process can be very brutal and feel like a lot of shuffling around / re-work. 🤗

sandrina-p commented 10 months ago

Yeah... This type of process can be very brutal and feel like a lot of shuffling around / re-work. 🤗

The 2nd most brutal thing is terrible UX of answering comments in GitHub xD