Skip to content
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

Position v3 (Dynamic position) #140

Open
wants to merge 35 commits into
base: main
Choose a base branch
from
Open

Conversation

codewithgun
Copy link
Contributor

No description provided.

@codewithgun codewithgun changed the title Feat/dynamic position Position v3 (Dynamic position) Dec 24, 2024
@codewithgun codewithgun force-pushed the feat/dynamic-position branch from a0c7262 to 90f1968 Compare December 24, 2024 03:57
@codewithgun codewithgun mentioned this pull request Dec 26, 2024
@codewithgun codewithgun force-pushed the feat/dynamic-position branch 9 times, most recently from 1b89b2d to a7dde02 Compare December 31, 2024 10:04
@codewithgun codewithgun force-pushed the feat/dynamic-position branch 2 times, most recently from 94e8345 to 83c2a05 Compare January 8, 2025 08:16
@codewithgun codewithgun force-pushed the feat/dynamic-position branch from 51aff30 to facad74 Compare January 13, 2025 17:40
@codewithgun codewithgun marked this pull request as ready for review January 14, 2025 10:17
- `getPairPubkeyIfExists` added new `baseFeePowerFactor` parameter
- `getAllPresetParameters` now return `presetParameter` and `presetParameter2` accounts instead of only `presetParameter`
- Rename `LBCLMM_PROGRAM_IDS` to `DLMM_PROGRAM_IDS`
- `createPermissionLbPair` added new `protocolFeeBps` parameter
Copy link
Contributor

Choose a reason for hiding this comment

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

do we need this function in sdk? caused it is only used by admin

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed


### Added

- Admin function `initialize_token_badge` to initialize token badge for token 2022
Copy link
Contributor

Choose a reason for hiding this comment

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

ditto^

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Was thinking to stop support rust cli in the program repo. This will be the main version moving forward.

}

#[derive(Parser, Debug)]
pub struct DownloadUserPoolFilesParams {
Copy link
Contributor

Choose a reason for hiding this comment

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

Why we need this function?

Copy link
Contributor Author

@codewithgun codewithgun Jan 15, 2025

Choose a reason for hiding this comment

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

Not needed. But putting it just for this PR to allow reviewer able to dump and run the test personally for v2 to v3 (dynamic position) test. Will remove it in next PR.

The test is here

// 1. Dump the account: target/debug/cli download-user-pool-files --wallet-key <wallet key> --override-wallet-key bossj3JvwiNK7pvjr149DqdtJxf2gdygbcmEPTkb2F1 --pool-key 6StaAqeVjKUPTgMLmcLmdpmUjoYHRcZ6uViLdWNdUghM --output-path $(readlink -f ./artifacts/jlp-hubsol-6StaAqeVjKUPTgMLmcLmdpmUjoYHRcZ6uViLdWNdUghM)

@codewithgun codewithgun force-pushed the feat/dynamic-position branch 2 times, most recently from f474a7e to a845e4d Compare February 4, 2025 05:12
@codewithgun codewithgun force-pushed the feat/dynamic-position branch from 02320c9 to 0cb73dd Compare February 10, 2025 07:49
Sign up for free to 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.

3 participants