-
Notifications
You must be signed in to change notification settings - Fork 59
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
For a storage deal proposal that is rejected, include rejection reason in client response #182
Conversation
Codecov Report
@@ Coverage Diff @@
## master #182 +/- ##
=======================================
Coverage 68.61% 68.61%
=======================================
Files 37 37
Lines 1991 1991
=======================================
Hits 1366 1366
Misses 528 528
Partials 97 97
Continue to review full report at Codecov.
|
Action(func(deal *storagemarket.ClientDeal, state storagemarket.StorageDealStatus) error { | ||
deal.Message = xerrors.Errorf("deal wasn't accepted (State=%d)", state).Error() | ||
Action(func(deal *storagemarket.ClientDeal, state storagemarket.StorageDealStatus, reason string) error { | ||
deal.Message = xerrors.Errorf("deal failed: (State=%d) %s", state, reason).Error() |
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.
Changed this because "deal rejected" is already in the message and the status itself is storagemarket.StorageDealFailing
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.
yea agree.
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
Action(func(deal *storagemarket.ClientDeal, state storagemarket.StorageDealStatus) error { | ||
deal.Message = xerrors.Errorf("deal wasn't accepted (State=%d)", state).Error() | ||
Action(func(deal *storagemarket.ClientDeal, state storagemarket.StorageDealStatus, reason string) error { | ||
deal.Message = xerrors.Errorf("deal failed: (State=%d) %s", state, reason).Error() |
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.
yea agree.
64b3dfb
to
43e4b9d
Compare
Closes #176