Skip to content

fix(p2p-orderbook): rollback c6c3cc1, f5f0b18 to restore stability#2631

Merged
shamardy merged 3 commits intodevfrom
rollback-2605-2580
Oct 6, 2025
Merged

fix(p2p-orderbook): rollback c6c3cc1, f5f0b18 to restore stability#2631
shamardy merged 3 commits intodevfrom
rollback-2605-2580

Conversation

@shamardy
Copy link
Collaborator

@shamardy shamardy commented Oct 3, 2025

This PR rolls c6c3cc1 and f5f0b18 back to restore stability of the orderbook functionalities in dev branch. These commits will be added back to #2626 to have one place for duplicate orders and orderbook validation among other p2p network fixes to be tested together.

I think this should be merged without squashing.

@shamardy shamardy marked this pull request as draft October 3, 2025 05:51
@shamardy shamardy marked this pull request as ready for review October 3, 2025 06:01
@mariocynicys
Copy link
Collaborator

did the two revertions create any conflicts? Otherwise i can safely skip reviewing them.

@shamardy
Copy link
Collaborator Author

shamardy commented Oct 3, 2025

did the two revertions create any conflicts?

Only the first reversion, conflict was in cargo.toml only.

Copy link
Collaborator

@mariocynicys mariocynicys left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm ✅

@shamardy shamardy force-pushed the rollback-2605-2580 branch from 98abd42 to 6f6098d Compare October 6, 2025 06:30
@shamardy shamardy merged commit 4025b8c into dev Oct 6, 2025
19 of 25 checks passed
@shamardy shamardy deleted the rollback-2605-2580 branch October 6, 2025 07:34
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.

2 participants