Skip to content

Make id option required for command spo list webhook get and set #3860

@milanholemans

Description

@milanholemans

The command spo list webhook get has an option id which is marked as optional in the docs. Also in the command logic the id is an optional property.
This should be a required property because the code assumes that it is always filled in.

What has to be adjusted?

  • Make the option required in the docs
  • Remove ID option from telemetry
  • Make the option required in our code

Sort of same problem for spo list webhook set. In the docs this option is marked as optional while it should be required. The code is fine. For this command we can also specify that the expirationDateTime cannot be more than 6 months in the future.

Metadata

Metadata

Type

No type

Projects

No projects

Milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions