Skip to content

fix: process all contract classes in storeBroadcastedIndividualFunctions (A-683)#21686

Merged
alexghr merged 1 commit intomerge-train/spartanfrom
phil/a-683-storebroadcastedindividualfunctions-returns-after-first
Mar 17, 2026
Merged

fix: process all contract classes in storeBroadcastedIndividualFunctions (A-683)#21686
alexghr merged 1 commit intomerge-train/spartanfrom
phil/a-683-storebroadcastedindividualfunctions-returns-after-first

Conversation

@PhilWindle
Copy link
Collaborator

Remove early return in for...of loop that caused only the first contract class's functions to be stored when multiple classes had broadcasts in the same block.

Fixes https://linear.app/aztec-labs/issue/A-683

@PhilWindle PhilWindle marked this pull request as ready for review March 17, 2026 17:40
@alexghr alexghr enabled auto-merge (squash) March 17, 2026 17:59
@alexghr alexghr merged commit 8738399 into merge-train/spartan Mar 17, 2026
24 of 28 checks passed
@alexghr alexghr deleted the phil/a-683-storebroadcastedindividualfunctions-returns-after-first branch March 17, 2026 18:06
AztecBot pushed a commit that referenced this pull request Mar 17, 2026
…ons (A-683) (#21686)

Remove early return in for...of loop that caused only the first contract
class's functions to be stored when multiple classes had broadcasts in
the same block.

Fixes https://linear.app/aztec-labs/issue/A-683
@AztecBot
Copy link
Collaborator

✅ Successfully backported to backport-to-v4-staging #21694.

ludamad pushed a commit that referenced this pull request Mar 17, 2026
…ons (A-683) (#21686)

Remove early return in for...of loop that caused only the first contract
class's functions to be stored when multiple classes had broadcasts in
the same block.

Fixes https://linear.app/aztec-labs/issue/A-683
ludamad added a commit that referenced this pull request Mar 17, 2026
BEGIN_COMMIT_OVERRIDE
fix: HA deadlock for last block edge case (#21690)
fix: process all contract classes in storeBroadcastedIndividualFunctions
(A-683) (#21686)
END_COMMIT_OVERRIDE
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants