fix: scaled orders wallet migration #77
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add scaled_orders table migration to migrateDefaultVenueWalletLocked: - Clone scaled_orders from old wallet to new wallet - Delete scaled_orders associated with old wallet This ensures the scaled order audit trail is preserved when a user changes their Hyperliquid wallet address, matching the behavior of hyperliquid_submissions and hyperliquid_status_history tables. Fixes Bug #1: Missing scaled_orders in Wallet Migration
51cf587
into
codex/extend-cmd/recomma-to-handle-multiple-venues
2 checks passed
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Added CloneScaledOrdersToWallet call in migrateDefaultVenueWalletLocked to copy scaled_orders from old wallet to new wallet
Added DeleteScaledOrdersForWallet call to remove scaled_orders from old wallet
Ensures scaled order audit trail is preserved during wallet changes