Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Evaluate attrs options hash #223

Merged
merged 2 commits into from
Jun 22, 2016
Merged

Evaluate attrs options hash #223

merged 2 commits into from
Jun 22, 2016

Conversation

offirgolan
Copy link
Collaborator

The options hash (model.get('validations.attrs.username.options')) is currently useless when a validator options is a function. This fixes that by having it contain the latest cached options.

Resolves ember-bootstrap/ember-bootstrap-cp-validations#4

@offirgolan offirgolan changed the title Validator options hash is now computed and evaluated instead of containing functions Evaluate attrs options hash Jun 22, 2016
@offirgolan offirgolan merged commit d682193 into master Jun 22, 2016
@offirgolan offirgolan deleted the option-changes branch June 22, 2016 17:36
offirgolan added a commit that referenced this pull request Jun 27, 2016
* master:
  [Feature] Warning Validators (#226)
  Evaluate attrs options hash (#223)
  chore(package): update ember-try to version 0.2.4 (#221)
  Fix for data validator with format (#218)
  chore(package): update ember-cli to version 2.6.2 (#217)
  chore(package): update ember-cli-moment-shim to version 2.0.0 (#216)
  chore(package): update ember-cli-blanket to version 0.9.5 (#214)
  Add import for getOwner to README example (#215)

# Conflicts:
#	addon/validations/result-collection.js
#	addon/validations/result.js
#	tests/integration/validations/factory-general-test.js
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Models with calculated presence are always required
1 participant