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

Added argument checks - for better error display. #475

Conversation

mercy-wumi
Copy link
Contributor

@mercy-wumi mercy-wumi commented Sep 12, 2024

Summary of Changes

Added argument checks to update movie review function in order to display user-friendly error message.
Remove description argument passed to AddMovieReview and UpdateMovieReview instruction macro as it is not used in the Accounts struct.

@mikemaccana
Copy link
Collaborator

Thanks @mercy-wumi ! While you didn't win this particular lesson, there's a lot of good stuff in this PR and we appreciate it!

@mikemaccana mikemaccana closed this Oct 7, 2024
@mikemaccana mikemaccana reopened this Oct 7, 2024
@mikemaccana mikemaccana merged commit 0ebfd1e into solana-foundation:main Oct 7, 2024
4 checks passed
adpthegreat pushed a commit to adpthegreat/developer-content that referenced this pull request Oct 23, 2024
)

* Added argument checks - for better error display.

* Update content/courses/onchain-development/anchor-pdas.md

* Update content/courses/onchain-development/anchor-pdas.md

---------

Co-authored-by: Mike MacCana <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants