Skip to content
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

Bf/proposal fetch #3281

Merged
merged 4 commits into from
Jun 6, 2024
Merged

Bf/proposal fetch #3281

merged 4 commits into from
Jun 6, 2024

Conversation

bfish713
Copy link
Collaborator

@bfish713 bfish713 commented Jun 6, 2024

Closes #3240

This PR:

  • Stores most recent proposals in memory
  • Adds interface for sequencer to store proposals we send
  • Calls this interface before sending out a proposal
  • Adds the stored proposals to the HotshotInitializer so they can be loaded from the sequencer on startup

This PR does not:

Key places to review:

New interface and init code

@bfish713 bfish713 marked this pull request as ready for review June 6, 2024 14:54
@bfish713 bfish713 requested review from rob-maron, shenkeyao, jparr721 and ss-es and removed request for elliedavidson and rob-maron June 6, 2024 14:54
jparr721
jparr721 previously approved these changes Jun 6, 2024
crates/types/src/consensus.rs Outdated Show resolved Hide resolved
@bfish713 bfish713 requested review from ss-es and jparr721 June 6, 2024 18:40
@bfish713 bfish713 merged commit f8b27f9 into main Jun 6, 2024
24 checks passed
@bfish713 bfish713 deleted the bf/proposal-fetch branch June 6, 2024 19:38
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.

[CATCHUP] - Store Proposals we Send Out
3 participants