Skip to content

Conversation

@aznrayizzle
Copy link
Contributor

added back public vs. permissioned page

@awrichar
Copy link
Contributor

Looks like you need to add signoff for DCO.

This looks fine to me, but might be worth a check with @peterbroadhurst to confirm if this is close enough to the current committed architecture. This doc looks like it was written while FFTM was still evolving, so could need some updating.

@codecov-commenter
Copy link

codecov-commenter commented Jan 17, 2023

Codecov Report

Merging #1152 (e8fb0c6) into main (9d1ae2a) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1152   +/-   ##
=======================================
  Coverage   99.98%   99.98%           
=======================================
  Files         305      305           
  Lines       19966    19966           
=======================================
  Hits        19964    19964           
  Misses          1        1           
  Partials        1        1           
Impacted Files Coverage Δ
internal/apiserver/server.go 99.08% <0.00%> (-0.92%) ⬇️
internal/events/webhooks/webhooks.go 100.00% <0.00%> (+0.93%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Signed-off-by: aznrayizzle <ray.chen@kaleido.io>
@awrichar awrichar merged commit 7f008a9 into hyperledger:main Jan 19, 2023
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.

3 participants