-
Notifications
You must be signed in to change notification settings - Fork 16
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Update old_list_neurons_service.certified.idl.js (#729)
# Motivation The purpose of `old_list_neurons_service.certified.idl.js` is to have the request type be supported by Ledger HW ICP app version 2.4.9. The hardware wallet does care about the response type so it is safe to update. We want to include `visibility` specifically. # Changes Recreate `old_list_neurons_service.certified.idl.js` the same way it was created the first time as documented by the comment at the top: ``` // This file was created manually by taking governance.certified.idl.js and // removing everything that isn't needed for `list_neurons` and then removing // 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. ``` The result is that the `visibility` field was added in addition to field names now being wrapped in `''`. # Tests Used manually with nns-dapp and was able to 1. show neurons from a hardware wallet, 2. see `visibility` in the response. # Todos - [ ] Add entry to changelog (if necessary). not necessary
- Loading branch information
Showing
1 changed file
with
54 additions
and
52 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters