governance-ui icon indicating copy to clipboard operation
governance-ui copied to clipboard

Disable 'Send It' button for Proposal comment when user has no voting power and show tooltip

Open SebastianBor opened this issue 3 years ago • 2 comments

SebastianBor avatar Feb 05 '22 21:02 SebastianBor

~~About to make a PR for this issue but this line is preventing me from pushing. I haven't touched that part of the codebase yet. Any advice?~~

Argument of type '[number, number, { [x: string]: {}; }, BN, number, boolean, BN, { accounts: { registrar: PublicKey; voter: PublicKey; voterAuthority: PublicKey; ... 10 more ...; rent: PublicKey; }; }]' is not assignable to parameter of type '[...unknown[], Context<Accounts<{ name: "registrar"; isMut: false; isSigner: false; } | { name: "voter"; isMut: true; isSigner: false; } | { name: "voterAuthority"; isMut: false; isSigner: false; } | { name: "voterWeightRecord"; isMut: true; isSigner: false; } | ... 8 more ... | { ...; }>>]'.
  Type at position 7 in source is not compatible with type at position 1 in target.
    The types of 'accounts' are incompatible between these types.
      Type '{ registrar: PublicKey; voter: PublicKey; voterAuthority: PublicKey; voterWeightRecord: PublicKey; vault: PublicKey; depositToken: PublicKey; ... 7 more ...; rent: PublicKey; }' is not assignable to type 'Accounts<{ name: "registrar"; isMut: false; isSigner: false; } | { name: "voter"; isMut: true; isSigner: false; } | { name: "voterAuthority"; isMut: false; isSigner: false; } | { name: "voterWeightRecord"; isMut: true; isSigner: false; } | ... 8 more ... | { ...; }>'.
        Object literal may only specify known properties, and 'tokenAuthority' does not exist in type 'Accounts<{ name: "registrar"; isMut: false; isSigner: false; } | { name: "voter"; isMut: true; isSigner: false; } | { name: "voterAuthority"; isMut: false; isSigner: false; } | { name: "voterWeightRecord"; isMut: true; isSigner: false; } | ... 8 more ... | { ...; }>'.ts(2345)

Fixed. Had to update my packages. PR incoming

RaymondDashWu avatar Feb 06 '22 07:02 RaymondDashWu

Merged. @SebastianBor can you close this issue?

RaymondDashWu avatar Apr 13 '22 20:04 RaymondDashWu