Skip to content

Reid carlberg patch 2 #10275

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

Merged
merged 6 commits into from
May 20, 2025
Merged

Conversation

cindylay
Copy link
Contributor

Category

  • Content fix
  • New article
  • Example checked item (delete this line)

DELETE THIS LINE BEFORE SUBMITTING | For the above list, an empty checkbox is [ ] as in [SPACE]. A checked checkbox is [x] with no space between the brackets. Use the PREVIEW tab at the top right to preview the rendering before submitting your issue.

Related issues

  • fixes #issuenumber
  • partially #issuenumber
  • mentioned in #issuenumber

DELETE THIS LINE BEFORE SUBMITTING | If this fixes (aka: closes) or references an issue, please reference it here. This helps maintaining the issue list as it will (1) link the PR to the issue & (2) automatically close the issue when this PR is merged in.

What's in this Pull Request?

DELETE THIS LINE BEFORE SUBMITTING | Please describe the changes in this PR. Sample description or details around bugs which are being fixed.

Submission guidelines

  • !!IMPORTANT!! - All submissions must complete the baseline sections included in this template. Ignoring or deleting this template may result in closing the issue with the label type:incomplete-submission.
  • Follow our guidance on How To Create Good Pull Requests.
  • Target the main branch of this repo.
  • When changing a page, ensure you update the ms.date front matter wih the current date in the format MM/DD/YYYY.
  • Review all build checks and address the automated errors, warnings, and suggestions.
  • NOTE: The live site is based on the live branch. Site owners periodically refresh live branch from the main branch so merged PRs won't immediately appear on the live site. Please be patient to see your changes appear on the live site.

DELETE THIS SECTION BEFORE SUBMITTING

ReidCarlberg and others added 6 commits April 22, 2025 14:42
The -OwningApplicationId is a UUID format, and in the example it was just 8 random characters, which wasn't clear.
Syntax for an array of hosts was wrong.
@cindylay cindylay merged commit 091c7ac into SharePoint:main May 20, 2025
1 of 2 checks passed
Copy link
Contributor

Learn Build status updates of commit fc6049e:

⚠️ Validation status: warnings

File Status Preview URL Details
docs/embedded/development/declarative-agent/spe-da-adv.md ⚠️Warning View Details

docs/embedded/development/declarative-agent/spe-da-adv.md

  • Line 20, Column 150: [Warning: hard-coded-locale - See documentation] Link 'https://learn.microsoft.com/en-us/microsoft-365-copilot/' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
  • Line 96, Column 155: [Warning: bookmark-not-found - See documentation] Cannot find bookmark '#rag--retrieval-augmented-generation-' in 'embedded/development/declarative-agent/spe-da-adv.md'.
  • Line 20, Column 150: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://learn.microsoft.com/en-us/microsoft-365-copilot/' will be broken in isolated environments. Replace with a relative link.

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:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants