Skip to content

feat: add neuron info command#70

Merged
mducroux merged 2 commits intomainfrom
mducroux/neuron-add-get-info-cmd
Jan 6, 2026
Merged

feat: add neuron info command#70
mducroux merged 2 commits intomainfrom
mducroux/neuron-add-get-info-cmd

Conversation

@mducroux
Copy link
Copy Markdown
Contributor

Add the neuron info command to display full neuron information:

NeuronInfo {
    neuronId: NeuronId;
    dissolveDelaySeconds: bigint;
    recentBallots: Array<BallotInfo>;
    neuronType: Option<NeuronType>;
    createdTimestampSeconds: bigint;
    state: NeuronState;
    joinedCommunityFundTimestampSeconds: Option<bigint>;
    retrievedAtTimestampSeconds: bigint;
    votingPower: bigint;
    ageSeconds: bigint;
    fullNeuron: Option<Neuron>;
    visibility: Option<NeuronVisibility>;
}

@mducroux mducroux requested a review from a team as a code owner December 19, 2025 08:55
@mducroux
Copy link
Copy Markdown
Contributor Author

Note: on the ledger wallet, when signing the transaction, the transaction type is: "List Own Neurons" which is incorrect. I am not sure how to fix that though.

Copy link
Copy Markdown
Collaborator

@ielashi ielashi left a comment

Choose a reason for hiding this comment

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

LGTM, thanks! I think it's ok that it shows "List Own Neurons". If we want to change that, that would be on Zondax's side.

Copy link
Copy Markdown
Contributor

@peterpeterparker peterpeterparker left a comment

Choose a reason for hiding this comment

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

Randomly spotted this PR, sharing if interesting. Feel free to ignore and not answer.

Comment thread src/index.ts Outdated
Comment thread src/utils.ts
@mducroux mducroux requested a review from a team January 6, 2026 15:59
@mducroux mducroux merged commit 665c8e5 into main Jan 6, 2026
5 checks passed
@mducroux mducroux deleted the mducroux/neuron-add-get-info-cmd branch January 6, 2026 16:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants