-
Notifications
You must be signed in to change notification settings - Fork 284
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 examples of managing attribute mappings #9418
Conversation
Learn Build status updates of commit 6361632: ✅ Validation status: passed
For more details, please refer to the build report. For any questions, please:
|
Learn Build status updates of commit 6467720: ✅ Validation status: passed
For more details, please refer to the build report. For any questions, please:
|
#sign-off |
@ArvindHarinder1 , this is the public mirror of the docs for external contributions. Can you please open this PR in the microsoft-graph-docs authoring repo, so we can follow our content workflow? Thanks. |
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.
Hi @ArvindHarinder1 - Please move these changes to the private repo = https://github.com/microsoftgraph/microsoft-graph-docs, then close this PR. Some key validations and the preview feature aren't available in this public mirror and therefore errors in your file aren't getting caught at this stage.
Apply the same review suggestions to the branch you open in the private repo.
api-reference/v1.0/api/synchronization-synchronizationschema-update.md
Outdated
Show resolved
Hide resolved
# [JavaScript](#tab/javascript) | ||
[!INCLUDE [sample-code](../includes/snippets/javascript/update-synchronizationschema-javascript-snippets.md)] | ||
[!INCLUDE [sdk-documentation](../includes/snippets/snippets-sdk-documentation-link.md)] | ||
|
||
--- |
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.
Delete this - it'll be added by the snippets generator
api-reference/v1.0/api/synchronization-synchronizationschema-update.md
Outdated
Show resolved
Hide resolved
@@ -42,7 +42,7 @@ In the request body, supply the [synchronizationSchema](../resources/synchroniza | |||
|
|||
If successful, returns a `204 No Content` response code. It doesn't return anything in the response body. | |||
|
|||
## Example | |||
## Example 1 | |||
|
|||
##### Request |
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.
Reduce this header and the "Response" to H4
api-reference/v1.0/api/synchronization-synchronizationschema-update.md
Outdated
Show resolved
Hide resolved
api-reference/v1.0/api/synchronization-synchronizationschema-update.md
Outdated
Show resolved
Hide resolved
api-reference/v1.0/api/synchronization-synchronizationschema-update.md
Outdated
Show resolved
Hide resolved
}, | ||
"targetAttributeName":"timezone" | ||
} | ||
], |
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.
Stray comma
api-reference/v1.0/api/synchronization-synchronizationschema-update.md
Outdated
Show resolved
Hide resolved
api-reference/v1.0/api/synchronization-synchronizationschema-update.md
Outdated
Show resolved
Hide resolved
…pdate.md Co-authored-by: Faith Moraa Ombongi <ombongi.moraa.fe@gmail.com>
…pdate.md Co-authored-by: Faith Moraa Ombongi <ombongi.moraa.fe@gmail.com>
…pdate.md Co-authored-by: Faith Moraa Ombongi <ombongi.moraa.fe@gmail.com>
…pdate.md Co-authored-by: Faith Moraa Ombongi <ombongi.moraa.fe@gmail.com>
…pdate.md Co-authored-by: Faith Moraa Ombongi <ombongi.moraa.fe@gmail.com>
…pdate.md Co-authored-by: Faith Moraa Ombongi <ombongi.moraa.fe@gmail.com>
…pdate.md Co-authored-by: Faith Moraa Ombongi <ombongi.moraa.fe@gmail.com>
…pdate.md Co-authored-by: Faith Moraa Ombongi <ombongi.moraa.fe@gmail.com>
Learn Build status updates of commit 0807b95: ✅ Validation status: passed
For more details, please refer to the build report. For any questions, please:
|
Learn Build status updates of commit 980836c:
|
File | Status | Preview URL | Details |
---|---|---|---|
api-reference/v1.0/api/synchronization-synchronizationschema-update.md | Details |
api-reference/v1.0/api/synchronization-synchronizationschema-update.md
- Line 188, Column 1: [Warning: invalid-tab-group - See documentation]
Tab group with different tab id set.
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.
For any questions, please:
- Try searching the learn.microsoft.com contributor guides
- Post your question in the Learn support channel
Instructions: Add any supporting information, such as a description of the PR changes, here.
Note
The following guidance is for Microsoft employees only. Community contributors can ignore this message; our content team will manage the status.
After you've created your PR, expand this section for tips and additional instructions.
#feedback-addressed
to the pull request.For more information, see the Content review process summary.