[Excel] (Platform limits) Add conditional access limit workaround #802
Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR updates the platform limits documentation to include information about conditional access authentication context limitations and clarifies the scope to include error messages.
- Updated title to reflect that the document covers error messages in addition to platform limits and requirements
- Added documentation about authentication context settings that prevent Office Scripts from running in Excel on the web
- Renamed the TOC entry to "Errors and platform limits" for better clarity
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| docs/toc.yml | Updated navigation label to include "Errors" in the platform limits section |
| docs/testing/platform-limits.md | Added authentication context limitation details and updated metadata to reflect expanded scope |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
Learn Build status updates of commit 5aba498: ✅ Validation status: passed
For more details, please refer to the build report. |
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
PoliCheck Scan ReportThe following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 issues. Other issues are also a high priority. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans. ✅ No issues foundMore information about PoliCheckInformation: PoliCheck | Severity Guidance | Term |
1 similar comment
PoliCheck Scan ReportThe following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 issues. Other issues are also a high priority. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans. ✅ No issues foundMore information about PoliCheckInformation: PoliCheck | Severity Guidance | Term |
|
Learn Build status updates of commit 5dc8328: ✅ Validation status: passed
For more details, please refer to the build report. |
Co-authored-by: Sam Ramon <15154970+samantharamon@users.noreply.github.com>
PoliCheck Scan ReportThe following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 issues. Other issues are also a high priority. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans. ✅ No issues foundMore information about PoliCheckInformation: PoliCheck | Severity Guidance | Term |
|
Learn Build status updates of commit 3458eb5: ✅ Validation status: passed
For more details, please refer to the build report. |
This PR adds a new note about Conditional Access policy limitations, and adjusts the name and description of the article to encourage discoverability -- customers have not been finding the error messages listed in the article.