-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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 details on Intune for mobile-app #27749
base: master
Are you sure you want to change the base?
Conversation
Based on latest customer questions, let's enrich our documentation to answer them.
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!
Preview links (active after the
|
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.
Hey David, I left some suggestions to match our style guide. Let me know if you have any questions 🙂
Co-authored-by: Janine Chan <[email protected]>
Hi @janine-c just applied your suggestions 🙇🏼♂️ |
@@ -65,6 +65,11 @@ Permissions are automatically added when configuring the application: | |||
| Microsoft Graph | `User.Read` | Sign in and read user profile | Delegated | | |||
| Microsoft Mobile Application Management | `DeviceManagementManagedApps.ReadWrite` | Read and Write the User's App Management device | Delegated | | |||
|
|||
**Notes:** | |||
- The mobile app only uses these two permissions. When granting consent you may see more permissions, because the mobile app shares the same Microsoft Entra application as the web app and the [Microsoft Teams Integration](https://docs.datadoghq.com/integrations/microsoft_teams). If you're not using it, you can revoke those permissions from the **Permissions** tab on the page in [step 3](#step-3-grant-admin-consent-for-your-organization). | |||
```suggestion |
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.
Just need to remove this line and then we're good to go!
```suggestion |
Based on latest customer questions, let's enrich our documentation to answer them.
What does this PR do? What is the motivation?
Merge instructions
Merge readiness:
Merge queue is enabled in this repo. To have it automatically merged after it receives the required reviews, create the PR (from a branch that follows the
<yourname>/description
naming convention) and then add the following PR comment:Additional notes