Skip to content

fix/update_creation_msg_liquidate_position - #296

Merged
aarmoa merged 1 commit into
masterfrom
fix/update_creation_msg_liquidate_position
Jan 18, 2024
Merged

fix/update_creation_msg_liquidate_position#296
aarmoa merged 1 commit into
masterfrom
fix/update_creation_msg_liquidate_position

Conversation

@aarmoa

Copy link
Copy Markdown
Contributor
  • Updated the logic to create the MsgLiquidatePosition message
  • Added a example script on how to broadcast a MsgLiquidatePositions message

…ew example script on how to broadcast a MsgLiquidatePosition
@codecov

codecovBot commented Jan 18, 2024

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (1ef9948) 72.97% compared to head (1606c4e) 72.97%.

Additional details and impacted files
@@ Coverage Diff @@## master #296 +/- ##
=======================================
Coverage 72.97% 72.97% =======================================
Files 48 48 Lines 3404 3404 Branches 303 303 =======================================
Hits 2484 2484 Misses 839 839 Partials 81 81 

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@nicolasbaumnicolasbaum left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Change looks good. Addition of optional argument for liquidator order allows full usage of message fields.

@aarmoa
aarmoa merged commit 6c47732 into masterJan 18, 2024
@aarmoa
aarmoa deleted the fix/update_creation_msg_liquidate_position branch January 18, 2024 20:54
Sign up for freeto 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.

2 participants

@aarmoa@nicolasbaum