-
Notifications
You must be signed in to change notification settings - Fork 144
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
[Feature] Filecoin.SyncSubmitBlock RPC. #4246
Conversation
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
src/rpc/methods/sync.rs
Outdated
use crate::blocks::GossipBlock; | ||
use crate::libp2p::{IdentTopic, NetworkMessage, PUBSUB_BLOCK_STR}; |
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.
nit: with the rest of the imports
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.
What did I miss?
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.
Ah, I can see now! not clear without seeing the full diff.
Summary of changes
Changes introduced in this pull request:
Filecoin.SyncSubmitBlock
RPC endpoint.Reference issue to close (if applicable)
Work on #4245
Other information and links
Needed for miner. There needs to be a follow-up that introduces all the sanity checks, the description can be found in the referenced issue.
Change checklist