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

Define IDL instructions for the interest bearing mint extension #45

Merged
merged 7 commits into from
Nov 14, 2024

Conversation

qapquiz
Copy link
Contributor

@qapquiz qapquiz commented Nov 13, 2024

This PR update the Interest Bearing Mint extension in IDL with test.
This pull request was created for https://app.gib.work/bounties/cadbce12-707f-4bde-967f-386cd83f5c2d in an attempt to solve a bounty #20 . Payment for the bounty is immediately sent to the contributor after merge.

@qapquiz qapquiz marked this pull request as draft November 13, 2024 15:03
@qapquiz qapquiz marked this pull request as ready for review November 13, 2024 16:57
@qapquiz
Copy link
Contributor Author

qapquiz commented Nov 13, 2024

let me fix merge conflict in a bit

@qapquiz
Copy link
Contributor Author

qapquiz commented Nov 13, 2024

Done 💯

Copy link
Member

@lorisleiva lorisleiva left a comment

Choose a reason for hiding this comment

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

Thank you, that's a good PR! Just a few comments and LGTM.

program/idl.json Outdated Show resolved Hide resolved
@qapquiz
Copy link
Contributor Author

qapquiz commented Nov 14, 2024

Okay sure. I will fix it by tonight (GMT+7). Thanks for the review ❤️

@lorisleiva
Copy link
Member

Thank you! No rush. 👌

- change `isSigner` from `true` to `either` in account rateAuthority in
updateRateInterestBearingMint
- use `createMint` function instead then update instruction in
updateInterestBearingMint test
@qapquiz
Copy link
Contributor Author

qapquiz commented Nov 14, 2024

Fixed as review. Thanks for the review lorisleiva ❤️

Copy link
Member

@lorisleiva lorisleiva left a comment

Choose a reason for hiding this comment

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

Thank you!

@lorisleiva lorisleiva merged commit 060f7cc into solana-program:main Nov 14, 2024
5 checks passed
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.

2 participants