jonjamz / blaze-forms

Dead easy reactive forms with validation (Meteor).
https://atmospherejs.com/templates/forms
MIT License
113 stars 11 forks source link

Changed parent form search to allow arbitrary parent distances. #100

Open paulbalomiri opened 8 years ago

paulbalomiri commented 8 years ago

solution to #99 (just the distance < 6 removal part)

jonjamz commented 8 years ago

First commit seems cool, I need to look at it more in a bit. The second commit has something I don't recognize--data.string--is this something you're using personally?

paulbalomiri commented 8 years ago

sorry, no data schema i'm going to correct that (i did, but did not push, sorry)

paulbalomiri commented 8 years ago

These changes allow for customizing of the store. From the comments i understand that you are planning a store package?

The next changes will make arrays available, as my current implementation hacks around the problem by using subforms for each array member.

The problem: one validation context for all array.$.member I think the way to go forward is to introduce data paths and schema paths where data paths are resolved into schema paths using info from the template instance.

Update: The schemaContext of each element should validate only the array element it represents

Part of it proposed in #98