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

feat(connector-fabric): contract deployment Fabric 2.x #841

Merged
merged 1 commit into from
Apr 27, 2021

Conversation

petermetz
Copy link
Contributor

Primary change:

Add support for deploying contracts to Fabric 2.x ledgers
Languages supported (and covered by tests):

  • golang
  • javascript
  • typescript

It is assumed that the server that is running the Fabric
ledger containers have the required dependencies to build
the contracts.
In the case of the tests this is provided by the Cactus
Fabric All-In-One container image which has NodeJS and Go
installed on it.

Secondary change(s):

Breaking API change: renamed the contract deployment endpoint
that supports Fabric 1.4.x so that:

  • the default implementation of it now assumes Fabric 2.x
  • there is still a compatible version of the endpoint for 1.4.x

Fixes #654

Signed-off-by: Peter Somogyvari peter.somogyvari@accenture.com

cc: @RafaelAPB

Primary change:
--------------

Add support for deploying contracts to Fabric 2.x ledgers
Languages supported (and covered by tests):
- golang
- javascript
- typescript

It is assumed that the server that is running the Fabric
ledger containers have the required dependencies to build
the contracts.
In the case of the tests this is provided by the Cactus
Fabric All-In-One container image which has NodeJS and Go
installed on it.

Secondary change(s):
-------------------

Breaking API change: renamed the contract deployment endpoint
that supports Fabric 1.4.x so that:
- the default implementation of it now assumes Fabric 2.x
- there is still a compatible version of the endpoint for 1.4.x

Fixes hyperledger-cacti#654

Signed-off-by: Peter Somogyvari <peter.somogyvari@accenture.com>
Copy link
Contributor

@jonathan-m-hamilton jonathan-m-hamilton left a comment

Choose a reason for hiding this comment

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

LGTM

@petermetz petermetz merged commit 139a8ed into hyperledger-cacti:main Apr 27, 2021
@petermetz petermetz deleted the feat-654 branch April 27, 2021 04:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file Fabric
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feat(connector-fabric): contract deployment to 2.x ledgers
3 participants