Skip to content

Commit

Permalink
Update comment in old_list_neurons_service.certified.idl.js (#730)
Browse files Browse the repository at this point in the history
# Motivation

I updated the comment to be more accurate when updating the service file
in #729 but then forgot to commit
these changes.

# Changes

Improve the comment's accuracy.

# Tests

N/A

# Todos

- [ ] Add entry to changelog (if necessary).
not necessary
  • Loading branch information
dskloetd authored Oct 11, 2024
1 parent 74e0223 commit a11a884
Showing 1 changed file with 5 additions and 4 deletions.
9 changes: 5 additions & 4 deletions packages/nns/candid/old_list_neurons_service.certified.idl.js
Original file line number Diff line number Diff line change
@@ -1,9 +1,10 @@
// This file was created manually by taking governance.certified.idl.js and
// removing everything that isn't needed for `list_neurons` and then removing
// `include_empty_neurons_readable_by_caller` from `ListNeurons`.
// The Ledger hardware wallet doesn't support the
// `include_empty_neurons_readable_by_caller` field, even when it's not set, so
// we use this service for compatibility with the hardware wallet.
// all fields except `neuron_ids` and `include_neurons_readable_by_caller` from
// `ListNeurons`.
// The Ledger hardware wallet app verion 2.4.9 doesn't support the newer fields,
// even when they are optional and not set, so we use this service for
// compatibility with the hardware wallet.
export const idlFactory = ({ IDL }) => {
const NeuronId = IDL.Record({ 'id' : IDL.Nat64 });
const Followees = IDL.Record({ 'followees' : IDL.Vec(NeuronId) });
Expand Down

0 comments on commit a11a884

Please sign in to comment.