From fd7cd5a125f0be47ed3b758190f8a1c9fa9d7c53 Mon Sep 17 00:00:00 2001 From: jasonz-dfinity <133917836+jasonz-dfinity@users.noreply.github.com> Date: Tue, 4 Feb 2025 08:42:22 -0800 Subject: [PATCH] chore(nns): Modify the comment on `include_empty_neurons_readable_by_caller` (#3740) After changing the default value in a previous [PR](https://github.com/dfinity/ic/pull/3612), the comment on the field should also be updated. --- rs/nns/governance/api/src/ic_nns_governance.pb.v1.rs | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/rs/nns/governance/api/src/ic_nns_governance.pb.v1.rs b/rs/nns/governance/api/src/ic_nns_governance.pb.v1.rs index e2ab19fa20b..b8ab14d17e4 100644 --- a/rs/nns/governance/api/src/ic_nns_governance.pb.v1.rs +++ b/rs/nns/governance/api/src/ic_nns_governance.pb.v1.rs @@ -3496,10 +3496,9 @@ pub struct ListNeurons { pub include_neurons_readable_by_caller: bool, /// Whether to also include empty neurons readable by the caller. This field only has an effect /// when `include_neurons_readable_by_caller` is true. If a neuron's id already exists in the - /// `neuron_ids` field, then the neuron will be included in the response regardless of the value of - /// this field. Since the previous behavior was to always include empty neurons readable by caller, - /// if this field is not provided, it defaults to true, in order to maintain backwards - /// compatibility. Here, being "empty" means 0 stake, 0 maturity and 0 staked maturity. + /// `neuron_ids` field, then the neuron will be included in the response regardless of the value + /// of this field. The default value is false (i.e. `None` is treated as `Some(false)`). Here, + /// being "empty" means 0 stake, 0 maturity and 0 staked maturity. #[prost(bool, optional, tag = "3")] pub include_empty_neurons_readable_by_caller: Option, /// If this is set to true, and a neuron in the "requested list" has its