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

Fix/teams compliance recording policy #3754

Merged

Conversation

sandrola
Copy link
Contributor

@sandrola sandrola commented Oct 4, 2023

Pull Request (PR) description

Add CIM Instance to MSFT_TeamsComplianceRecordingPolicy to be able to compare with configured policy

This Pull Request (PR) fixes the following issues

ykuijs
ykuijs previously requested changes Nov 10, 2023
Copy link
Member

@ykuijs ykuijs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Few small comments

CHANGELOG.md Outdated Show resolved Hide resolved
@andikrueger
Copy link
Collaborator

@sandrola Thanks for your PR and the work you put into it! Any chance you could resolve conflicts and address Yorick's comments?

@pkernevez
Copy link
Contributor

As the original issue #3754 has been closed, I think this PR should be close

Copy link
Collaborator

@andikrueger andikrueger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just one minor remark and a question for @NikCharlebois

@NikCharlebois
Copy link
Collaborator

Can we please merge and update the PR with the latest changes? Thanks

@ykuijs
Copy link
Member

ykuijs commented Mar 27, 2024

There are still some conflicts with the Dev branch that need to get resolved before we can merge this PR. Could you have another look?

@sandrola sandrola requested a review from ykuijs May 16, 2024 06:43
@FabienTschanz
Copy link
Contributor

@sandrola Could you please update or close the PR? Thank you.

@NikCharlebois
Copy link
Collaborator

As we approach our next Major release (Oct 2nd), we need to make confirm this is still a valid PR and update it to resolve any merge conflicts.

@sandrola
Copy link
Contributor Author

sandrola commented Oct 1, 2024

The PR is still valid and I just updated it with dev.

@NikCharlebois NikCharlebois dismissed ykuijs’s stale review October 2, 2024 12:17

Did the PR Review

@NikCharlebois NikCharlebois merged commit 3f4e1e2 into microsoft:Dev Oct 2, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

MSFT_TeamsComplianceRecordingPolicy attribute not exported correctly
6 participants