Update based on customer feedback#1929
Update based on customer feedback#1929rodrigoffonseca wants to merge 8 commits intoMicrosoftDocs:mainfrom
Conversation
New PR to superseed 1925 adn 1926
Updated image file names and date format in the SAP IAG integration documentation.
Updated image syntax for clarity and consistency in the SAP integration documentation.
|
@rodrigoffonseca : Thanks for your contribution! The author(s) and reviewer(s) have been notified to review your proposed change. |
|
Learn Build status updates of commit 32f058d: 💡 Validation status: suggestions
docs/id-governance/media/entitlement-management-sap-integration/sap-iag-mapping.png
For more details, please refer to the build report. Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them. |
PRMerger Results
|
|
Can you review the proposed changes? IMPORTANT: When the changes are ready for publication, adding a #label:"aq-pr-triaged" |
There was a problem hiding this comment.
Pull request overview
Updates the SAP IAG integration documentation to reflect customer feedback by clarifying the required attribute mappings and adding new screenshots to illustrate the correct configuration.
Changes:
- Updated the prerequisites section to instruct mapping Microsoft Entra
objectIdto SAP’s Global User ID (userUuid) and added corresponding screenshots. - Added guidance for adding the SCIM enterprise
managerattribute mapping. - Refreshed the article date and added two new PNG assets.
Reviewed changes
Copilot reviewed 1 out of 3 changed files in this pull request and generated 9 comments.
| File | Description |
|---|---|
| docs/id-governance/entitlement-management-sap-integration.md | Updates prerequisites guidance and embeds new screenshots for the attribute mapping configuration. |
| docs/id-governance/media/entitlement-management-sap-integration/new-user-id-mapping.png | New screenshot showing the objectId → userUuid mapping. |
| docs/id-governance/media/entitlement-management-sap-integration/sap-iag-mapping.png | New screenshot showing the overall attribute mapping state. |
| ms.subservice: entitlement-management | ||
| ms.topic: how-to #Required; leave this attribute/value as-is | ||
| ms.date: 11/20/2025 | ||
| ms.date: 03/25/2026 |
There was a problem hiding this comment.
If this article was created or substantially modified with AI assistance, add the required ai-usage front matter entry (for example, ai-usage: ai-assisted) to comply with the repo’s AI disclosure guidance.
| ms.date: 03/25/2026 | |
| ms.date: 03/25/2026 | |
| ai-usage: ai-assisted |
| SAP Cloud Identity Services instance that is already integrated with Microsoft Entra for: | ||
| - User Provisioning: See, [Configure SAP Cloud Identity Services for automatic user provisioning with Microsoft Entra ID](../identity/saas-apps/sap-cloud-platform-identity-authentication-provisioning-tutorial.md) | ||
| - In your attribute mapping, set Microsoft Entra ObjectId as mapped to the username by selecting *EDIT* in the username line and set the Source and Target attribute as: | ||
| - In your attribute mapping, add a new mapping to sync Microsoft Entra ObjectId to be mapped as SAP Global User ID. This is required to allow the correct object look up between the two systems. |
There was a problem hiding this comment.
Use the single-word form “lookup” here for consistency and correct usage.
| - In your attribute mapping, add a new mapping to sync Microsoft Entra ObjectId to be mapped as SAP Global User ID. This is required to allow the correct object look up between the two systems. | |
| - In your attribute mapping, add a new mapping to sync Microsoft Entra ObjectId to be mapped as SAP Global User ID. This is required to allow the correct object lookup between the two systems. |
| - User Provisioning: See, [Configure SAP Cloud Identity Services for automatic user provisioning with Microsoft Entra ID](../identity/saas-apps/sap-cloud-platform-identity-authentication-provisioning-tutorial.md) | ||
| - In your attribute mapping, set Microsoft Entra ObjectId as mapped to the username by selecting *EDIT* in the username line and set the Source and Target attribute as: | ||
| - In your attribute mapping, add a new mapping to sync Microsoft Entra ObjectId to be mapped as SAP Global User ID. This is required to allow the correct object look up between the two systems. | ||
| - Click *Add New Mapping* and set the attribute mapping as below: |
There was a problem hiding this comment.
UI elements should be formatted in bold (not italics) per Learn style and to match the rest of this article (for example, +Add, Save, etc.). Consider changing Add New Mapping to bold UI text.
| :::image type="content" source="media/entitlement-management-sap-integration/new-user-id-mapping.png" alt-text="Screenshot of setting manager attribute for SAP integration."::: | ||
|
|
||
| - Add a mapping for the manager attribute. This allows manager information to be synchronized from Microsoft Entra to SAP Cloud Identity Services. | ||
| - Click *Add New Mapping* and set the attribute mapping as below: |
There was a problem hiding this comment.
UI elements should be formatted in bold (not italics) per Learn style and to match the rest of this article. Consider changing Add New Mapping to bold UI text here as well.
| :::image type="content" source="media/entitlement-management-sap-integration/manager-attribute.png" alt-text="Screenshot of setting manager attribute for SAP integration." lightbox="media/entitlement-management-sap-integration/manager-attribute.png"::: | ||
| - Target Attribute: urn:ietf:params:scim:schemas:extension:sap:2.0:User:userUuid | ||
|
|
||
| :::image type="content" source="media/entitlement-management-sap-integration/new-user-id-mapping.png" alt-text="Screenshot of setting manager attribute for SAP integration."::: |
There was a problem hiding this comment.
The alt text for this image references the manager attribute, but the screenshot is about mapping objectId to the SAP user UUID/Global User ID. Update the alt text to describe what’s actually shown.
| :::image type="content" source="media/entitlement-management-sap-integration/new-user-id-mapping.png" alt-text="Screenshot of setting manager attribute for SAP integration."::: | |
| :::image type="content" source="media/entitlement-management-sap-integration/new-user-id-mapping.png" alt-text="Screenshot of attribute mapping from objectId to SAP userUuid (Global User ID) in SAP Cloud Identity Services."::: |
|
|
||
| :::image type="content" source="media/entitlement-management-sap-integration/new-user-id-mapping.png" alt-text="Screenshot of setting manager attribute for SAP integration."::: | ||
|
|
There was a problem hiding this comment.
The :::image directive isn’t indented as part of the list item (and has extra leading spaces), which can break list formatting in Markdown. Indent the image block to the same nesting level as the surrounding bullet content.
| :::image type="content" source="media/entitlement-management-sap-integration/new-user-id-mapping.png" alt-text="Screenshot of setting manager attribute for SAP integration."::: | |
| :::image type="content" source="media/entitlement-management-sap-integration/new-user-id-mapping.png" alt-text="Screenshot of setting manager attribute for SAP integration."::: | |
| - Source Attribute: manager | ||
| - Target Attribute: urn:ietf:params:scim:schemas:extension:enterprise:2.0:User:manager |
There was a problem hiding this comment.
The nested list indentation under the manager mapping is inconsistent (the Source/Target Attribute lines are indented differently than other nested bullets). Adjust indentation so the Source/Target bullets are clearly nested under the “Click Add New Mapping…” bullet (typically one additional 4-space level).
| - Source Attribute: manager | |
| - Target Attribute: urn:ietf:params:scim:schemas:extension:enterprise:2.0:User:manager | |
| - Source Attribute: manager | |
| - Target Attribute: urn:ietf:params:scim:schemas:extension:enterprise:2.0:User:manager |
| - Click *Add New Mapping* and set the attribute mapping as below: | ||
| - Source Attribute: manager | ||
| - Target Attribute: urn:ietf:params:scim:schemas:extension:enterprise:2.0:User:manager | ||
| - Your Attribute Mapping should look like this when the pre-requesites are complete: |
There was a problem hiding this comment.
Typo: “pre-requesites” should be “prerequisites”.
| - Your Attribute Mapping should look like this when the pre-requesites are complete: | |
| - Your Attribute Mapping should look like this when the prerequisites are complete: |
|
|
||
| :::image type="content" source="media/entitlement-management-sap-integration/sap-iag-mapping.png" alt-text="Screenshot of attribute mapping for SAP integration."::: | ||
|
|
There was a problem hiding this comment.
This :::image directive should be indented to remain within the bullet list (and avoid rendering issues). Right now it’s aligned at the left with extra spaces, which can terminate the list unexpectedly.
As aligned with Sri Ponnada, proposing changes to the doc based on feedback, this update superseeds previous PR sent earlier and canceled due to errors.