Initial changset validation state about ember-changeset-validations HOT 9 OPEN

@poteto The validation that is proposed works nice but it expects you to define changeset outside of templates. When (changeset data validation) is used I'm not able to find a way how to do it and I have to modify your module.

It is enough to extend function changeset([model, validationMap]) with changset.validate(). This should be configurable, so adding a new attribute (e.g. initValidation with false as a default) looks sensible. If I will prepare PR, are you willing to merge it?

from ember-changeset-validations.

Could you call validate on the init of whatever it is that's being instantiated, like in your snippet above?

from ember-changeset-validations.

@poteto For sure, but it does limit building validated changesets in a template. Should this be a default behaviour in your mind? Perhaps an option that can be passed to the constructor?

from ember-changeset-validations.

@marxsk That's what I ultimately ended up doing and just setting a controller or component property for the template to bind to the pre-validated changeset object.

from ember-changeset-validations.

@poteto could we bump this issue ? This is badly needed.
I have the same issue and using validate() right after the Changeset has been initialised is not the proper solution I think.
For instance, by doing that it forces all the error messages to show up on the first render of a form.

from ember-changeset-validations.

@MrChocolatine I often use the suggested pattern with much success. I call validate() right after initializing the Changeset, but then I hide validation errors in the form until other events occur such as an individual input blur (show errors for that field) or the user tries to submit the form (show errors for all fields). I really like the flexibility and control that this approach gives me.

from ember-changeset-validations.

To me it sounds more like an incorrect implementation. We should not to have to do this manually and a first validation should occur internally, when the Changeset is created. Or at least via an option as already suggested.

from ember-changeset-validations.

@MrChocolatine Appreciate the input! I certainly think an optional initValidation boolean approach is worthwhile and will help give users visibility into the initial state of the changeset without hooking into the JS side of things. I generally wait until user input to perform any of this logic (submit/blur) but I can see that limited use cases may require this API. I created an issue in ember-changeset to track and implement.

poteto/ember-changeset#451

from ember-changeset-validations.

Thank you @snewcomer 👍 .

from ember-changeset-validations.