Skip to content
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

Remove defining default properties from commands that are not returning a list #6432

Open
26 tasks
Adam-it opened this issue Oct 16, 2024 · 1 comment
Open
26 tasks

Comments

@Adam-it
Copy link
Contributor

Adam-it commented Oct 16, 2024

We should use defaultProperties method only when the command is of list type so returns more than one item. It should rather not be used in get, add or any other commands.

We could fix it up in the following cases:

  • entra user guest add
  • flow get
  • flow environment get
  • flow run get
  • pa app get
  • pa environment get
  • planner bucket add
  • planner plan add
  • planner plan get
  • planner plan set
  • planner task checklistitem add
  • pp aibuildmodel get
  • pp card get
  • pp chatbot get
  • pp dataverse table get
  • pp environment get
  • pp solution get
  • pp solution publisher get
  • spo group member add
  • spo page copy
  • spo page get
  • tenant serviceannouncement health get
  • todo list get
  • todo task get
  • viva engage message add
  • viva engage message get
@Adam-it Adam-it added enhancement needs peer review Needs second pair of eyes to review the spec or PR labels Oct 16, 2024
@milanholemans milanholemans added help wanted and removed needs peer review Needs second pair of eyes to review the spec or PR labels Oct 17, 2024
@SmitaNachan
Copy link
Contributor

Can I work on it?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants