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

fix(validation): merge field default options before init validation. #351

Merged
merged 1 commit into from
Dec 17, 2016
Merged

fix(validation): merge field default options before init validation. #351

merged 1 commit into from
Dec 17, 2016

Conversation

aitboudad
Copy link
Member

What kind of change does this PR introduce? Bug fix
Fixed #349

@codecov-io
Copy link

codecov-io commented Dec 17, 2016

Current coverage is 91.98% (diff: 100%)

Merging #351 into master will not change coverage

@@             master       #351   diff @@
==========================================
  Files            16         16          
  Lines           749        749          
  Methods         141        141          
  Messages          0          0          
  Branches        197        197          
==========================================
  Hits            689        689          
  Misses           60         60          
  Partials          0          0          

Powered by Codecov. Last update ea9a04f...7c0c6de

@aitboudad aitboudad merged commit ce2a589 into ngx-formly:master Dec 17, 2016
@aitboudad aitboudad deleted the issue_349 branch December 17, 2016 14:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Repeat Section [Validation not being applied to investmentDate]
4 participants