-
Notifications
You must be signed in to change notification settings - Fork 330
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 : tenant report settings set - closes #6247 #6548
base: main
Are you sure you want to change the base?
New command : tenant report settings set - closes #6247 #6548
Conversation
Thanks @NishkalankBezawada, we'll try to review it ASAP. |
Hi @NishkalankBezawada, I ran the tests that we usually do on PR's. There's a couple of issue though, or so it appears. Could you like at them and fix them in your code? You may want to run Let me know if you have questions. |
Hey @martinlingstuyl Thanks for reviewing. Since this is my first contribution to the repository, I might have missed a few things. I will surely check on this in the evening, and send in for your review. Thanks, |
Yes it's me 😄! You can always check our contribution documentation, which is quite good. Also when it's about unit tests. If you've got any questions, let me know 🚀 |
Hey @martinlingstuyl, This pretty much helped me. Will be sending a new review soon 🚀 Thanks, |
Hey @martinlingstuyl Can you review it once again. Thanks, |
New command : tenant report settings set - closes #6247
Explanation
This command can be used to update (adminReportSettings) tenant-level settings for Microsoft 365 reports.
Please refer to the Graph documentation page.
Usage
m365 tenant report settings set [options]
test screens
Below has set
--hideUserInformation
tofalse
below is the screen from the Admin center
Below has set
--hideUserInformation
totrue
below is the screen from the Admin center
Thanks,
Nish