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 meeting attendancereport list #3907

Closed
appieschot opened this issue Oct 25, 2022 · 4 comments
Closed

New command: m365 teams meeting attendancereport list #3907

appieschot opened this issue Oct 25, 2022 · 4 comments

Comments

@appieschot
Copy link
Member

appieschot commented Oct 25, 2022

Usage

m365 teams meeting attendancereport list

Description

Lists all attendance reports for a given meeting

Options

Option Description
-u, --userId [userId] The id of the user, omit to list attendance reports for current signed in user. Use either id, userName or email, not not all.
-n, --userName [userName] The name of the user, omit to list attendance reports for current signed in user. Use either id, userName or email, not not all.
--email [email] The email of the user, omit to list attendance reports for current signed in user. Use either id, userName or email, not not all.
-m, --meetingId <meetingId> The Id of the meeting

Examples

Lists all attendance reports made for the current signed in user and Microsoft Teams meeting with given id

m365 teams meeting attendancereport list --meetingId _MSo1N2Y5ZGFjYy03MWJmLTQ3NDMtYjQxMy01M2EdFGkdRWHJlQ_

Lists all attendance reports made for the garthf@contoso.com and Microsoft Teams meeting with given id

m365 teams meeting attendancereport list --userName garthf@contoso.com --meetingId MSo1N2Y5ZGFjYy03MWJmLTQ3NDMtYjQxMy01M2EdFGkdRWHJlQ

Additional Info

https://learn.microsoft.com/en-us/graph/api/meetingattendancereport-list?view=graph-rest-beta&tabs=http

We will need a check to make sure userName is required when they are signed in with app permissions.
⚠️ Make sure to handle error for meetings hosted in a channel as channel meetings are not supported
⚠️ Uses additional permissions, make sure to list them in PR OnlineMeetingArtifact.Read.All
⚠️ Add remark that the beta endpoint is subject to change

@appieschot appieschot added the needs peer review Needs second pair of eyes to review the spec or PR label Oct 25, 2022
@Jwaegebaert
Copy link
Contributor

Looks good @appieschot! Maybe an extra remark that the beta endpoint is subject to change.

@appieschot appieschot changed the title New command: m365 teams meeting attendance list New command: m365 teams meeting attendancereport list Oct 28, 2022
@appieschot appieschot added help wanted good first issue hacktoberfest Issue perfect for hacktoberfest and removed needs peer review Needs second pair of eyes to review the spec or PR labels Oct 28, 2022
@Jwaegebaert Jwaegebaert removed the hacktoberfest Issue perfect for hacktoberfest label Nov 2, 2022
@stijnhaerkenss
Copy link
Contributor

I can work on this

@Jwaegebaert
Copy link
Contributor

Awesome @stijnhaerkenss. All yours!

@milanholemans
Copy link
Contributor

Maybe we should also say in the docs: This method does not support channel meetings.

stijnhaerkenss added a commit to stijnhaerkenss/cli-microsoft365 that referenced this issue Nov 23, 2022
stijnhaerkenss added a commit to stijnhaerkenss/cli-microsoft365 that referenced this issue Nov 24, 2022
@milanholemans milanholemans added this to the v6.1 milestone Dec 4, 2022
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