Skip to content
This repository was archived by the owner on Jan 24, 2025. It is now read-only.

Program security - Account data matching lesson updated#440

Merged
mikemaccana merged 3 commits into
solana-foundation:mainfrom
0xCipherCoder:fix-program-security-account-data-matching
Oct 4, 2024
Merged

Program security - Account data matching lesson updated#440
mikemaccana merged 3 commits into
solana-foundation:mainfrom
0xCipherCoder:fix-program-security-account-data-matching

Conversation

@0xCipherCoder
Copy link
Copy Markdown
Contributor

Problem

Summary of Changes

  • Updated code snippets with the latest anchor version
  • Fixed content,
  • Fixed grammar and styling
  • Fixed as per guidelines

Fixes #
Unboxed PRs
Starter - solana-developers/account-data-matching#2
Solution - solana-developers/account-data-matching#3

@0xCipherCoder
Copy link
Copy Markdown
Contributor Author

@mikemaccana Please review this once. Let me know if any suggestions.

evaluates to true. Alternatively, you can use `has_one` to check that a target
account field stored on the account matches the key of an account in the
`Accounts` struct.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Weird but markdown highlighting goes wrong after this step. ('in' 'struct' etc are highlighted in text). Not sure if it's the PR or a github bug.

Comment thread content/courses/program-security/account-data-matching.md Outdated
@mikemaccana mikemaccana merged commit e019790 into solana-foundation:main Oct 4, 2024
adpthegreat pushed a commit to adpthegreat/developer-content that referenced this pull request Oct 23, 2024
…ation#440)

* Added fix for code snippets and content

* Update content/courses/program-security/account-data-matching.md

---------

Co-authored-by: Mike MacCana <mike.maccana@gmail.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants