-
Notifications
You must be signed in to change notification settings - Fork 29
Upgrade anchor #939
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
base: main
Are you sure you want to change the base?
Upgrade anchor #939
Conversation
| } | ||
|
|
||
| #[derive(AnchorSerialize, AnchorDeserialize, Clone, Debug)] | ||
| pub struct Ocr3ConfigInfoInput { |
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.
@jadepark-dev Does this change make sense? I copied it from your branch, I'm not sure if I should do the same to all the structs that I had to remove the AnchorSerialize and AnchorDeserialize?
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.
No, iirc it was only for Ocr3ConfigInfo. This could be a bug from Anchor. They changed AnchorSerialize, AnchorDeserialize to be implemented automatically from 0.30.x, but the compiler was complaining when it's used for input struct so I had to split the struct and give it those attributes.
But please keep in mind that the work was experimental, so it might have been fixed in a more recent version of Anchor or there might be a better pattern now!
|
Build contracts with latest version of anchor 0.31.0 running
make build-contracts.The go-bindings are not working right now, as a few changes need to be done.