-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Parameter help is not shown, even though the help info is available in the database #64
Comments
I can reproduce this on production by running I have confirmed that the intended data is available in the Azure DB with all the help info: (...)
{
"Aliases": "none",
"DefaultValue": "False",
"Description": "Indicates whether the command collects real time statistics.",
"Globbing": "false",
"IsDynamic": false,
"Name": "Realtime",
"ParameterSets": {
"__AllParameterSets": {
"IsMandatory": false,
"Position": -2147483648,
"ValueFromPipeline": false,
"ValueFromPipelineByPropertyName": false,
"ValueFromRemainingArguments": false,
"HelpMessage": null,
"HelpMessageBaseName": null,
"HelpMessageResourceId": null
}
},
"PipelineInput": "False",
"Position": "named",
"Required": "false",
"SwitchParameter": true,
"TypeName": "SwitchParameter"
},
(...) TODO: |
Ok local repro succeeded. Log says:
|
I was starting to suspect exactly this. The error says the json cannot be parsed to the target schema/type in C#. So I messed something up there. |
It seems there there are two problems here: one is that for some cmdlets, the parameter help is so big, it doesn't fit the 64k limit in an Azure Table storage property. This is why The other problem (where the parameter data does not fit the schema) I have not even touched yet. |
TODO: find out why sometimes the stored parameter data does not fit the json schema to load them |
Thanks @lucdekens for your patience on this one. Part one is now fixed as you can see: |
Just did a quick test with
It looks as if the explanation for the parameters is not expanded,
although it is available in the help.
Originally posted by @lucdekens in #63 (comment)
The text was updated successfully, but these errors were encountered: