Attention is currently required from: osmith.

fixeria has posted comments on this change by fixeria. ( 
https://gerrit.osmocom.org/c/osmo-hlr/+/37779?usp=email )

Change subject: subscriber-create-on-demand: rework configuration
......................................................................


Patch Set 1:

(1 comment)

File src/hlr_vty.c:

https://gerrit.osmocom.org/c/osmo-hlr/+/37779/comment/b998d013_4fa65ecf?usp=email
 :
PS1, Line 300:          vty_out(vty, " no subscriber-create-on-demand%s", 
VTY_NEWLINE);
> This is the default, I thought we omit these. […]
Well, I believe printing a setting explicitly is better than omitting values.
I expected to get some questions here, let's see what others think.



--
To view, visit https://gerrit.osmocom.org/c/osmo-hlr/+/37779?usp=email
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings?usp=email

Gerrit-MessageType: comment
Gerrit-Project: osmo-hlr
Gerrit-Branch: master
Gerrit-Change-Id: Ib553172655f83dad1ac0e0254615c8c207d79ca9
Gerrit-Change-Number: 37779
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: osmith <[email protected]>
Gerrit-Attention: osmith <[email protected]>
Gerrit-Comment-Date: Wed, 07 Aug 2024 12:35:42 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: osmith <[email protected]>

Reply via email to