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

GIB💎WORK Bounty - Pull Request Made To anchor #2751

Closed
wants to merge 5 commits into from

Conversation

SoSoDolo
Copy link

🤖 This pull request was created by https://app.gib.work/r/Ql3a0gKk/anchor
⚡️ Payment for bounty is immediately sent to the contributor after merge.

henrye and others added 5 commits February 23, 2023 11:11
…string, an array of numbers etc." This supports the 3 mentioned. Also, I included array of strings because I saw elsewhere an array of pubkeys could also be passed. If there are other types that may be expected that are encompassed in that etc. Let me know and I will update
Copy link

vercel bot commented Dec 29, 2023

@SoSoDolo is attempting to deploy a commit to the coral-xyz Team on Vercel.

A member of the Team first needs to authorize it.

@acheroncrypto acheroncrypto added duplicate This issue or pull request already exists ts labels Dec 30, 2023
@SoSoDolo
Copy link
Author

@acheroncrypto any reason this was marked as duplicate but there hasn't been an accepted PR? Just looking for some feedback

@acheroncrypto
Copy link
Collaborator

Hey, this looked like a spam to me since it's bringing out some random commits that was made in the past with autogenerated PR title and description:

image

There was also another PR(#2752) with the same PR title.

@SoSoDolo
Copy link
Author

@acheroncrypto Oh yeah not sure about the extra commits but the title is auto generated from GibWork so anybody that tries to claim will have this title I believe. Or is that not an actual bounty for this repo?

@acheroncrypto
Copy link
Collaborator

Or is that not an actual bounty for this repo?

There could be but I'm not the person who created it.

Also, we can reopen the PR if you fix the extra commits and add the correct types. The types in #2752 is not correct either, the Rust types are here:

#[derive(Debug, Clone, Serialize, Deserialize, PartialEq)]
#[serde(rename_all = "camelCase", tag = "kind")]
pub enum IdlSeed {
Const(IdlSeedConst),
Arg(IdlSeedArg),
Account(IdlSeedAccount),
}
#[derive(Debug, Clone, Serialize, Deserialize, PartialEq)]
#[serde(rename_all = "camelCase")]
pub struct IdlSeedAccount {
#[serde(rename = "type")]
pub ty: IdlType,
// account_ty points to the entry in the "accounts" section.
// Some only if the `Account<T>` type is used.
#[serde(skip_serializing_if = "Option::is_none")]
pub account: Option<String>,
pub path: String,
}
#[derive(Debug, Clone, Serialize, Deserialize, PartialEq)]
#[serde(rename_all = "camelCase")]
pub struct IdlSeedArg {
#[serde(rename = "type")]
pub ty: IdlType,
pub path: String,
}
#[derive(Debug, Clone, Serialize, Deserialize, PartialEq)]
#[serde(rename_all = "camelCase")]
pub struct IdlSeedConst {
#[serde(rename = "type")]
pub ty: IdlType,
pub value: serde_json::Value,
}

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
duplicate This issue or pull request already exists ts
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants