-
Notifications
You must be signed in to change notification settings - Fork 8.9k
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
Update msp.rst #691
Update msp.rst #691
Conversation
I learnt that Hyperledger Fabric does not provide custom OU support in profile section. But the documentation at https://hyperledger-fabric.readthedocs.io/en/master/msp.html#best-practices suggests in point number 2.1 that we can define policies which capture the custom OUs of the identity. Signed-off-by: Chintan Rajvir <chintanr97@gmail.com>
Signed-off-by: Chintan Rajvir <chintanr97@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@pamandrejko Need one more approval! |
@btl5037 Since we only require a single +1 here, what is preventing this from being merged? Jason's comments are marked resolved. |
@pamandrejko It says: "1 review requesting changes and 1 approving review by reviewers with write access." I did not understand this! |
I learnt that Hyperledger Fabric does not provide custom OU support in profile section. But the documentation at https://hyperledger-fabric.readthedocs.io/en/master/msp.html#best-practices suggests in point number 2.1 that we can define policies which capture the custom OUs of the identity.
Signed-off-by: Chintan Rajvir chintanr97@gmail.com
Type of change
Description
Additional details
Related issues