-
Notifications
You must be signed in to change notification settings - Fork 99
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
Add Connection to Teams and the first Meeting Policies Tests #518
Conversation
Thanks for this @weyCC81 One of the assumptions we make is that the user will not have the other modules like Teams and Exchange installed. So the part where Test-Cs... is called needs to be wrapped in a try..catch so it doesn't throw errors for users who don't have the module installed. |
Does the Build Error have to do something with me or? |
No you can ignore that build error. |
Thanks for the update @weyCC81 I've made some more updates after doing a few tests. I think it's starting to get there. I'm going to go ahead and merge into main, but will open an issue to keep track of and address these to make it easier for users to follow the test and the recommendation.
|
Add Connection
I have added the necessary test to connect to Teams without yet impacting the "All" scope.
In relation to
-SkipGraphConnect
I have adjusted some dependencies which need validation (see ToValidate and ToAdjust)Teams Meeting Policies
I have implemented the first Test, which are also in the M365 Secure Score for the area Teams.
PS: As this is one of my first commits, I am really open to feedback