-
Notifications
You must be signed in to change notification settings - Fork 124
Add issue #72 new type settings and added deviceRegistrationPolicy #111
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
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR adds a new "Settings" object type and a new device registration policy export to address issue #72, which requested the ability to export only settings without exporting all associated data (e.g., group settings without all groups).
Changes:
- Added a new "Settings" enum value to the ObjectType enum in alphabetical order
- Added the 'Settings' tag to four existing schema entries for settings-related endpoints
- Added a new schema entry for the
policies/deviceRegistrationPolicyendpoint
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| src/EntraExporterEnums.ps1 | Added 'Settings' to the ObjectType enum to support the new export type |
| src/Get-EEDefaultSchema.ps1 | Added 'Settings' tag to entitlement management, group, Teams, and SharePoint settings endpoints; added new deviceRegistrationPolicy endpoint |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
@microsoft-github-policy-service agree |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Test works for me.
PR for #72