-
Notifications
You must be signed in to change notification settings - Fork 5
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
Feature/prepare split #211
base: develop
Are you sure you want to change the base?
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
interface BlazeConfig { | ||
pool: PublicKey; | ||
bsolMint: PublicKey; | ||
} | ||
|
||
interface MarinadeConfig { |
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.
Is this in order to be able to remove the marinade sdk from our build?
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.
That could be a long-term idea. Right now it only reduces our dependency on it. i.e now we're able to derive the bsolTokenAccountAuthority without needing to have called initMarinade() first.
return [marinade, marinadeState]; | ||
} | ||
|
||
// TODO |
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.
can you add some more detail here of what is remaining?
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.
That shouldn't be there. I'll remove it now
TokenConfig
in SunriseStakeClient.makeBlaze()
,makeMarinade()
,getTokenConfig
thatinit()
now depends on.getMpDetails()
,getLpDetails()
, .... thatdetails()
now depends on.The external interface provided by SunriseStakeClient remains the same.