ember-changeset-validations icon indicating copy to clipboard operation
ember-changeset-validations copied to clipboard

Harmonize blank option

Open frykten opened this issue 3 years ago • 0 comments

Hello!

This is linked to my proposal in #317 .

So I changed the allowBlank's check for the date validator to Ember's util isEmpty, as per the ones in the ember-validators (and therefore the ones in the other validators of this addon, since all but date use ember-validators). However, while doing that, I saw that ember-validators does not use isEmpty for their dates (exceptionally) and instead make a custom check to only nullish values and empty strings (cf.). Maybe you would copy that, instead? I'm opening an issue on their side to ask them why this inconsistency.

Also, I updated the tests of the other addons to harmonize their checks and reflect the behaviours.

frykten avatar Oct 14 '21 13:10 frykten