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

Implemented channel creating operation with peer channel cli. #328

Merged
merged 6 commits into from
Oct 15, 2021

Conversation

zhuyuanmao
Copy link
Contributor

Create a channel for the current user, and then add all peer nodes from
the org to the channel. Finally, set the first peer node as the anchor
peer.

Signed-off-by: Yuanmao Zhu zhu.yuanmao18@gmail.com

Create a channel for the current user, and then add all peer nodes from
the org to the channel. Finally, set the first peer node as the anchor
peer.

Signed-off-by: Yuanmao Zhu <zhu.yuanmao18@gmail.com>
@zhuyuanmao zhuyuanmao requested a review from a team September 10, 2021 05:09
Yuanmao Zhu added 2 commits September 10, 2021 12:55
Signed-off-by: Yuanmao Zhu <zhu.yuanmao18@gmail.com>
Added validating checking for the number of orderers and peers.

Signed-off-by: Yuanmao Zhu <zhu.yuanmao18@gmail.com>
@@ -106,7 +198,7 @@ def create(self, request):
except ObjectDoesNotExist:
pass

@swagger_auto_schema(
@ swagger_auto_schema(
Copy link
Contributor

Choose a reason for hiding this comment

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

Do we need this extra space?

Yuanmao Zhu added 3 commits October 7, 2021 19:15
Signed-off-by: Yuanmao Zhu <zhu.yuanmao18@gmail.com>
Signed-off-by: Yuanmao Zhu <zhu.yuanmao18@gmail.com>
updating endpoint.

Signed-off-by: Yuanmao Zhu <zhu.yuanmao18@gmail.com>
@yeasy yeasy merged commit cb218f0 into hyperledger-cello:main Oct 15, 2021
@zhuyuanmao zhuyuanmao deleted the channel-creating branch March 24, 2022 04:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants