-
Notifications
You must be signed in to change notification settings - Fork 8
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(squid)!: squid object procurement #173
base: main
Are you sure you want to change the base?
Conversation
Code Coverage SummaryClick to see the summary
Click to see the extended report
|
#[error] | ||
const EWrongObject: vector<u8> = b"object passed did not match the requested object"; | ||
#[error] | ||
const ERemainingData: vector<u8> = b"remaining bcs data unexpected."; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
const ERemainingData: vector<u8> = b"remaining bcs data unexpected."; | |
const ERemainingData: vector<u8> = b"remaining bcs data unexpected"; |
const EWrongCoinType: vector<u8> = b"coin type mismatch."; | ||
#[error] | ||
const EWrongSwapType: vector<u8> = b"swap type mismatch."; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
const EWrongCoinType: vector<u8> = b"coin type mismatch."; | |
#[error] | |
const EWrongSwapType: vector<u8> = b"swap type mismatch."; | |
const EWrongCoinType: vector<u8> = b"coin type mismatch"; | |
#[error] | |
const EWrongSwapType: vector<u8> = b"swap type mismatch"; |
|
||
use squid::swap_info::SwapInfo; | ||
|
||
const SWAP_TYPE: u8 = 4; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
not a fan of partial declarations. define swap type enum in a separate type module and use it where needed?
swap_data.destroy(); | ||
} | ||
|
||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@@ -3,102 +3,107 @@ module squid::swap_info; | |||
use squid::coin_bag::{Self, CoinBag}; | |||
use sui::bcs; | |||
|
|||
public enum SwapStatus has copy, drop, store { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
change base of the PR to the other PR, so the diff is accurate
AXE-5765