-
Notifications
You must be signed in to change notification settings - Fork 153
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
fix: StateWaitMsg RPC API #3783
Conversation
Hmm, we should wait with merging this PR until #3784 is addressed. |
src/chain/store/chain_store.rs
Outdated
db: &DB, | ||
block_header: &BlockHeader, | ||
i: usize, | ||
) -> Result<Option<Receipt>, Error> | ||
where | ||
DB: Blockstore, | ||
{ | ||
// Try Receipt_v2 first |
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.
I'm not sure the logic for containing this should be in the chain_store
. Seems more like a shim
thingy.
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.
Fixed
@@ -32,6 +32,10 @@ impl ChainMessage { | |||
ChainMessage::Signed(msg) => msg.cid(), | |||
} | |||
} | |||
|
|||
pub fn equal_call(&self, other: &Self) -> bool { |
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.
Perhaps some docs? I have no idea what it does.
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.
Fixed
@LesnyRumcajs Resolved a merge conflict in |
Summary of changes
As part of #3639
Note:
return_dec
field is ignored inapi compare
tests, see #3801 (comment)Changes introduced in this pull request:
Filecoin.StateWaitMsg
RPC APIapi compare
testsReceipt
deserialization logicReference issue to close (if applicable)
Closes
Other information and links
Change checklist