-
Notifications
You must be signed in to change notification settings - Fork 7
Use Codama v1.0.0 #14
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
Conversation
buffalojoec
left a comment
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.
Looks great!!
| TProgramAddress extends Address = typeof SOLANA_CONFIG_PROGRAM_ADDRESS, | ||
| >( | ||
| input: StoreInput<TAccountConfigAccount>, | ||
| config?: { programAddress?: TProgramAddress } |
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.
Interesting, so one could use the config program client against a different config program (configurable program ID)?
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.
Yes! This is a late addition from @mcintyre94 (See PR). For instance, this allows someone to use the SPL Token JS client for the Token 2022 program (if someone was taking bloody ages to release a Token 2022 client for example 👀).
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.
I dig it!!
| "standard": "kinobi", | ||
| "version": "0.19.0" | ||
| "standard": "codama", | ||
| "version": "1.0.0" |
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.
Major version ?? 👀
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.
Major version indeed! It's showtime. 🌟
| "definedTypes": [ | ||
| { | ||
| "kind": "definedTypeNode", | ||
| "name": "configKeys", |
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.
Nice. I like how this is an IDL-defined type now, instead of in-lined.
See codama-idl/codama#234