Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

ivanvoznyakovsky
Copy link
Contributor

In our project we've come into a problem to skip validation of records returned by adapter. Eg. all records received from API should be treated as valid records. Passing noValidate: true to find/findAll was not respected cause property setter gets noValidate property value from Settable. Settable instance get's this property from schema definition, which is false by default. Hence no matter what value for noValidate we pass to find/findAll it does not affect validation.

@codecov-io
Copy link

codecov-io commented Feb 20, 2017

Codecov Report

Merging #446 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #446      +/-   ##
==========================================
+ Coverage   95.95%   95.96%   +<.01%     
==========================================
  Files          21       21              
  Lines        2498     2504       +6     
==========================================
+ Hits         2397     2403       +6     
  Misses        101      101
Impacted Files Coverage Δ
src/Collection.js 100% <100%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update edeceeb...de914ad. Read the comment docs.

Copy link
Member

@jmdobry jmdobry left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for this!

@jmdobry jmdobry added this to the 3.0.0 Stable milestone Feb 20, 2017
@jmdobry jmdobry merged commit fb065db into js-data:master Feb 20, 2017
@jmdobry jmdobry changed the title fix(Collection): respect noValidate override passed in to find/findAll when adding records fix(Collection): Add noValidate option to Collection#add Feb 20, 2017
jmdobry added a commit that referenced this pull request Feb 20, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants