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

New command: m365 teams team app list #4129

Closed
Tracked by #4116
garrytrinder opened this issue Nov 21, 2022 · 5 comments
Closed
Tracked by #4116

New command: m365 teams team app list #4129

garrytrinder opened this issue Nov 21, 2022 · 5 comments
Assignees
Milestone

Comments

@garrytrinder
Copy link
Member

garrytrinder commented Nov 21, 2022

Usage

m365 teams team app list

Description

List apps installed in the specified team

Options

Option Description
-i, --id [id] The Id of the Microsoft Team to list installed apps from. Specify either teamId or teamName but not both
-n, --name [name] The display name of the Microsoft Team to list installed apps from. Specify either teamId or teamName but not both

Examples

List applications installed in the specified Microsoft Team

m365 teams team app list --id 2eaf7dcd-7e83-4c3a-94f7-932a1299c844

List applications installed in the specified Microsoft Team

m365 teams team app list --name "Team Name"

Additional Info

The response should include both the teamsApp and teamsAppDefinition relationship data.

The implementation of this command is currently in the teams app list command.

Display properties should be id, displayName and distributionMethod.

@milanholemans
Copy link
Contributor

milanholemans commented Nov 21, 2022

Nice idea @garrytrinder
Few remarks:

  • your 2nd example uses id instead of name
  • let's not repeat the IDs etc in the example description. E.g. "List applications installed in a specific Microsoft Teams team"
  • let's specify deafult properties

Other than that, looking good! 👍

@garrytrinder
Copy link
Member Author

Good catch @milanholemans 👍🏻

I've updated incorrect option in the second example.

I've updated the example description, we should also look to update the examples in teams team get which I used to be consistent with existing commands.

I've added default properties. I always forget about stating these, so we should consider adding this to the new command template as a new field also.

@Jwaegebaert
Copy link
Contributor

Looks good @garrytrinder!

@MathijsVerbeeck
Copy link
Contributor

Pick me!

@Jwaegebaert
Copy link
Contributor

All yours!

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

Successfully merging a pull request may close this issue.

4 participants