Attention is currently required from: neels.
Patch set 4:Code-Review -1
2 comments:
Commit Message:
This is useful for user interaction, to prefill an input field that
indicates a valid input to modify to taste.
I think it looks odd in this patch because i later on separated the patch from the ParamSource patch […]
For parameters that actually have a sane default value, I agree it makes sense to have them stored here. But for those that do not, or even worse, where a default would have critical security implications, there should not be a default.
If you want to indicate an input format or an example, then that would be an "example content" or "input format" class member, but no a default value.
File pySim/esim/saip/personalization.py:
Patch Set #2, Line 278: default_value
that's true, it's a copy paste artifact from the random parameters... […]
there is never any use case for an all-zero ICCID. I don't see why we should add that in the code and why it should provide any advantage. If you want to specify length constraints, then that is what max_len and min_len are for. That is the actual "useful indicator for number of digits" as expressed by this class. Any code that wants to use that information in some kind of UI can use that.
To view, visit change 40094. To unsubscribe, or for help writing mail filters, visit settings.