Skip to content

Update control section#12614

Closed
TobiasAsk wants to merge 2 commits intoMicrosoftDocs:publicfrom
TobiasAsk:TobiasAsk-patch-1
Closed

Update control section#12614
TobiasAsk wants to merge 2 commits intoMicrosoftDocs:publicfrom
TobiasAsk:TobiasAsk-patch-1

Conversation

@TobiasAsk
Copy link
Copy Markdown

Refer to updated method of using the Graph API to control the activation of Feed.

Refer to updated method of using the Graph API to control the activation of Feed.
@learn-build-service-prod
Copy link
Copy Markdown
Contributor

Learn Build status updates of commit b2af246:

⚠️ Validation status: warnings

File Status Preview URL Details
microsoft-365/ms-feed/m365-feed.md ⚠️Warning Details

microsoft-365/ms-feed/m365-feed.md

  • Line 53, Column 133: [Warning: hard-coded-locale - See documentation] Link 'https://learn.microsoft.com/en-us/graph/api/resources/microsoftapplicationdataaccesssettings?view=graph-rest-beta' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
  • Line 53, Column 133: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://learn.microsoft.com/en-us/graph/api/resources/microsoftapplicationdataaccesssettings?view=graph-rest-beta' will be broken in isolated environments. Replace with a relative link.
  • Line 53, Column 133: [Suggestion: preserve-view-not-set - See documentation] You've pinned this link to a specific version of content with the view parameter. It's recommended not to pin a version unless that version is A) not the default view and B) the context is about that version specifically. To proceed with pinning a version add the &preserve-view=true to the URL. Otherwise, remove the view parameter. URL: https://learn.microsoft.com/en-us/graph/api/resources/microsoftapplicationdataaccesssettings?view=graph-rest-beta

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:

@denisebmsft denisebmsft added the admin Microsoft 365 admin content label Aug 10, 2023
Comment thread microsoft-365/ms-feed/m365-feed.md Outdated
Copy link
Copy Markdown
Contributor

@JohanFreelancer9 JohanFreelancer9 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy review - @TobiasAsk, kindly commit the suggested changes. Many thanks.

@TobiasAsk
Copy link
Copy Markdown
Author

Thanks for your suggestions @JohanFreelancer9, I've applied them

@learn-build-service-prod
Copy link
Copy Markdown
Contributor

Learn Build status updates of commit c99b5ee:

✅ Validation status: passed

File Status Preview URL Details
microsoft-365/ms-feed/m365-feed.md ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

@yogkumgit
Copy link
Copy Markdown

@temanteleary Please merge this PR. Thanks

@yogkumgit
Copy link
Copy Markdown

@temanteleary Gentle reminder. Thanks

@v-jmathew v-jmathew self-assigned this Oct 23, 2025
@v-jmathew v-jmathew added BulkPRReview PRs undergoing bulk review BulkPRReview-RedirectedToPrivate PRs that need to be moved to a private repo for consideration as part of a bulk PR review. labels Oct 23, 2025
@v-jmathew
Copy link
Copy Markdown
Contributor

This PR has been open for an extended period of time and has conflicts. If these changes are still relevant, please re-submit them to https://github.com/MicrosoftDocs/microsoft-365-docs-pr/.

@v-jmathew v-jmathew closed this Oct 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

admin Microsoft 365 admin content BulkPRReview PRs undergoing bulk review BulkPRReview-RedirectedToPrivate PRs that need to be moved to a private repo for consideration as part of a bulk PR review. Keep open Microsoft submitter

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants