Skip to content
This repository has been archived by the owner on Apr 13, 2022. It is now read-only.

fix(core-model): Validate to accept convector moddels #46

Conversation

diestrin
Copy link
Contributor

@diestrin diestrin commented Feb 24, 2019

Proposed changes

Convector decorator @Validate() did not support ConvectorModels properly
Closes #45

Types of changes

What types of changes does your code introduce to Convector?

  • Bugfix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist

  • I have read the CONTRIBUTING doc
  • Lint and unit tests pass locally with my changes
  • All the commits have been squashed into a single commit following the conventional commits guide
  • All the commits are signed with git sign-off
  • I have added tests that prove my fix is effective or that my feature works
  • I have added necessary documentation (if appropriate)

Closes hyperledger-labs#45

* schema is a function, needs to be invoked
* update unit-tests for FlatConvectorModels
* fix wrong property on the convector model schema

Signed-off-by: Diego Barahona <diestrin@gmail.com>
@diestrin diestrin added bug Something isn't working core Core packages labels Feb 24, 2019
@diestrin diestrin requested a review from walmon February 24, 2019 00:42
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working core Core packages
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Passing a ConvectorModel to @Validate is not currently working
2 participants