Skip to content

Conversation

@manoranjith
Copy link

I have renamed PeerRejectedProposalError to PeerRejectedError, so that the same error type can be used when peer rejects an update as well.

The other option could be to use two error types. Request your feedback on this.

@manoranjith manoranjith force-pushed the named-errors-peer-rejected-update branch 7 times, most recently from be03ae0 to f878155 Compare March 10, 2021 14:07
@manoranjith manoranjith changed the title Extend PeerNotRespondingError for rejected updates Extend PeerRejectedProposalError for rejected updates Mar 10, 2021
Manoranjith added 2 commits March 10, 2021 20:13
- So that the same error type can be used when peer rejects an update or
  a proposal..

Signed-off-by: Manoranjith <ponraj.manoranjitha@in.bosch.com>
Signed-off-by: Manoranjith <ponraj.manoranjitha@in.bosch.com>
@manoranjith manoranjith force-pushed the named-errors-peer-rejected-update branch from f878155 to fc4bafb Compare March 10, 2021 14:43
@ggwpez ggwpez merged commit 6d9d074 into hyperledger-labs:dev Mar 11, 2021
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