Skip to content

Commit

Permalink
solana: clarify that additional accounts are optional
Browse files Browse the repository at this point in the history
  • Loading branch information
johnsaigle committed Feb 29, 2024
1 parent 96ab9d0 commit 48fceb7
Showing 1 changed file with 4 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@
//! instruction is able to invoke the program's admin instructions.
//!
//! The instruction needs to be encoded in the VAA payload, with all the
//! accounts. These accounts may be in any order and must include two additional placeholder accounts:
//! accounts. These accounts may be in any order and may include two additional placeholder accounts:
//! - [`OWNER`]: the program will replace this account with the governance PDA
//! - [`PAYER`]: the program will replace this account with the payer account
use anchor_lang::prelude::*;
Expand Down Expand Up @@ -128,9 +128,9 @@ impl From<AccountMeta> for Acc {
/// The VAA's payload contains an instruction and relevant Accounts that it requires. This program
/// performs verification of the VAA's contents and then performs a Cross Program Invocation if all
/// verification succeeds.
/// NOTE: `missing_owner_check` is disabled here: The VAA instruction is expected to contain
/// placeholder accounts with Pubkeys set to hard-coded values [OWNER] and [PAYER]. These keys are
/// overwritten. Because they are placeholders, the `owner` field is not important.
/// NOTE: `missing_owner_check` is disabled here: The VAA instruction may contain
/// placeholder accounts with Pubkeys set to hard-coded values [OWNER] and [PAYER].
/// These keys are overwritten. Because they are placeholders, the `owner` field is not important.
#[allow(unknown_lints)]
#[allow(missing_owner_check)]
pub fn governance<'info>(ctx: Context<'_, '_, '_, 'info, Governance<'info>>) -> Result<()> {
Expand Down

0 comments on commit 48fceb7

Please sign in to comment.