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

Added Places/Rooms #2379

Merged
merged 1 commit into from
May 3, 2024
Merged

Conversation

OfficialEsco
Copy link
Contributor

@OfficialEsco OfficialEsco commented Apr 29, 2024

The Place.ReadWrite.All and Place.Read.All API permission needs to be added

@KelvinTegelaar
Copy link
Owner

If you'd like to make the entire PR complete; in CIPP-API you can edit Modules\CIPPCore\Public\SAMManifest.json to include the permission, this means that permission will be detected in CPV refreshes, permissions check, and the default app will have it next time someone deploys :)

@JohnDuprey
Copy link
Collaborator

close for hotfix

@JohnDuprey JohnDuprey closed this May 2, 2024
@JohnDuprey JohnDuprey reopened this May 2, 2024
@KelvinTegelaar KelvinTegelaar merged commit fff9cf1 into KelvinTegelaar:dev May 3, 2024
6 of 11 checks passed
@OfficialEsco OfficialEsco deleted the PlacesPR branch May 3, 2024 09:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants