-
Notifications
You must be signed in to change notification settings - Fork 0
Solana: Execute proposal (only transfer) #70
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
Merged
Merged
Changes from all commits
Commits
Show all changes
31 commits
Select commit
Hold shift + click to select a range
90513c6
Add min_validations to Controller
GuidoDipietro 4004443
Use controller's min_validations on settler's create_intent
GuidoDipietro 04a39b9
Add tests for min_validations on Settler
GuidoDipietro ad78d1e
Fix incorrect assertion in tests
GuidoDipietro fba3c30
Scaffold execute_proposal body
GuidoDipietro 142117e
Implement execute_proposal (transfer intent)
GuidoDipietro 35a6545
Abstract execute_transfer out to function
GuidoDipietro fabea4e
Documents execute_transfer remaining_accounts transfer branch
GuidoDipietro 0e018a1
execute_proposal WIP
GuidoDipietro 53b2737
Improvements in execute_proposal
GuidoDipietro 1b7824c
WIP tests
GuidoDipietro a0ed7e2
Fix double borrow issue in Rust
GuidoDipietro 9ab7b69
Fix execute_proposal tests scaffold
GuidoDipietro 4d9ea5c
Implement execute_proposal happy path!
GuidoDipietro 7b1a75d
Implement some unhappy paths
GuidoDipietro e7c406d
Implement more unhappy paths
GuidoDipietro d1230de
Finish execute_proposal tests (first iteration)
GuidoDipietro 4468a9d
Fix clippy
GuidoDipietro 8fa0272
Cargo fmt
GuidoDipietro 5b0be6d
Add pay_solver_fees logic
GuidoDipietro 41c785c
Scaffold tests for pay_solver_fees
GuidoDipietro 8fa39bf
Implement happy path with pay_solver_fees
GuidoDipietro b3902ab
Add custom errors for pay_solver_fees
GuidoDipietro 50b09a9
Implement missing tests
GuidoDipietro 7d840b2
Run clippy and yarn lint, remove .skip
GuidoDipietro 8b20045
Merge branch 'solana/settler' into solana/execute-proposal
GuidoDipietro 723d852
Code review comments
GuidoDipietro 24540b8
Simplify before in function
GuidoDipietro 941e1ee
Remove nasty arrow functions from helper functions
GuidoDipietro 9e86fa6
Remove duplicated nested context
GuidoDipietro d5d2088
Nit
GuidoDipietro File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
Large diffs are not rendered by default.
Oops, something went wrong.
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
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
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
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
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
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
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
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
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
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,3 @@ | ||
| pub mod transfer; | ||
|
|
||
| pub use transfer::*; |
14 changes: 14 additions & 0 deletions
14
packages/svm/programs/settler/src/types/intent_data/transfer.rs
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,14 @@ | ||
| use anchor_lang::prelude::{borsh::BorshDeserialize, *}; | ||
|
|
||
| #[derive(BorshDeserialize)] | ||
| pub struct SvmTransfer { | ||
| pub token: Vec<u8>, | ||
| pub amount: u64, | ||
| pub recipient: Vec<u8>, | ||
| } | ||
|
|
||
| #[derive(BorshDeserialize)] | ||
| pub struct SvmTransferIntentData { | ||
| pub chain_id: u32, | ||
| pub transfers: Vec<SvmTransfer>, | ||
| } |
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,9 +1,11 @@ | ||
| pub mod eip712_domain; | ||
| pub mod intent_data; | ||
| pub mod intent_event; | ||
| pub mod op_type; | ||
| pub mod token_fee; | ||
|
|
||
| pub use eip712_domain::*; | ||
| pub use intent_data::*; | ||
| pub use intent_event::*; | ||
| pub use op_type::*; | ||
| pub use token_fee::*; |
Oops, something went wrong.
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.