Skip to content
This repository has been archived by the owner on Sep 4, 2024. It is now read-only.

Bucket support in vesting #21

Merged
merged 4 commits into from
Nov 28, 2023
Merged

Bucket support in vesting #21

merged 4 commits into from
Nov 28, 2023

Conversation

Adrena-Corto
Copy link

As titled. Consider the buckets for vest creation and claim. Add and update tests

@@ -29,6 +29,9 @@ pub async fn test_integration() {
tests_suite::staking::liquid_staking_overlap_remove_same_as_overlap().await;
tests_suite::staking::liquid_staking_overlap_remove_more_than_overlap().await;

tests_suite::vesting::vote().await;
Copy link
Author

Choose a reason for hiding this comment

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

This test suite wasn't used, I think it's a mistake? Seems it doesn't work though, so I'll comment it for now (unrelated to this PR)

Copy link

@adrena-orex adrena-orex left a comment

Choose a reason for hiding this comment

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

Looking good!

@@ -86,4 +86,6 @@ pub enum PerpetualsError {
PermissionlessOracleSignerMismatch,
#[msg("Signed message does not match instruction params")]
PermissionlessOracleMessageMismatch,
#[msg("The bucket does not contain enough token for reserving this allocation")]
InsuficientBucketReserve,

Choose a reason for hiding this comment

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

Insuficient Insufficient

);
}
}
cortex.update_bucket_minted_amount(params.bucket_name, params.amount)?;

Choose a reason for hiding this comment

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

clean!

@adrena-orex adrena-orex merged commit ea6bf40 into adx Nov 28, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants