You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
What is this task and why do we need to work on it?
Right now the quorum proposal recv task relies on calling the existing consensus functions, but isolates the call site so that way it can exist in an idempotent manner. This has created some issues due to the need to keep backwards compatibility with the consensus task. Instead, we will author our own new logic and only reuse functions that have limited scope.
What work will need to be done to complete this task?
Implement the handler for the QuorumProposalRecv event.
Exhaustively unit test all branching cases.
Handle the liveness check and vote.
Are there any other details to include?
We do not yet have a test for the liveness check, so we may need to align the implementation to the legacy consensus task for now.
What are the acceptance criteria to close this issue?
All tests pass.
Branch work will be merged to (if not the default branch)
No response
The text was updated successfully, but these errors were encountered:
What is this task and why do we need to work on it?
Right now the quorum proposal recv task relies on calling the existing consensus functions, but isolates the call site so that way it can exist in an idempotent manner. This has created some issues due to the need to keep backwards compatibility with the consensus task. Instead, we will author our own new logic and only reuse functions that have limited scope.
What work will need to be done to complete this task?
QuorumProposalRecv
event.Are there any other details to include?
We do not yet have a test for the liveness check, so we may need to align the implementation to the legacy consensus task for now.
What are the acceptance criteria to close this issue?
All tests pass.
Branch work will be merged to (if not the default branch)
No response
The text was updated successfully, but these errors were encountered: