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

support for multiple fabric transaction format #495

Merged
merged 17 commits into from
Nov 1, 2023
Merged

Conversation

adecaro
Copy link
Contributor

@adecaro adecaro commented Sep 14, 2023

This PR brings in the following:

  • Support for multiple Fabric transaction formats.
  • Removal of fmt.Errorf

@alexandrosfilios alexandrosfilios force-pushed the f-fabric-custom-tx branch 3 times, most recently from 5dff108 to 1fb22ab Compare September 19, 2023 08:35
@alexandrosfilios alexandrosfilios force-pushed the f-fabric-custom-tx branch 3 times, most recently from d65a8ff to bb91b83 Compare September 28, 2023 16:29
@alexandrosfilios alexandrosfilios force-pushed the f-fabric-custom-tx branch 5 times, most recently from 937da80 to 844f687 Compare October 6, 2023 09:55
adecaro and others added 6 commits October 9, 2023 14:53
Added public constructor for endorser transaction factory

Signed-off-by: Alexandros Filios <[email protected]>
Added rawRequest to the parameters of NewTransaction

Signed-off-by: Alexandros Filios <[email protected]>
@adecaro adecaro self-assigned this Oct 31, 2023
Copy link
Contributor

@alexandrosfilios alexandrosfilios left a comment

Choose a reason for hiding this comment

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

LGTM

@adecaro adecaro merged commit 90ce30c into main Nov 1, 2023
19 checks passed
@adecaro adecaro deleted the f-fabric-custom-tx branch November 1, 2023 08:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants