-
Notifications
You must be signed in to change notification settings - Fork 271
[All hosts](NAA SSO) Updates for NAA GA and WXP online support #5410
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
base: main
Are you sure you want to change the base?
Conversation
davidchesnut
commented
Oct 14, 2025
- Updates for NAA GA.
- Adds code for login hint required for Word, Excel, and PowerPoint on the web.
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 14a15ae: ✅ Validation status: passed
For more details, please refer to the build report. |
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 d868b84: ✅ Validation status: passed
For more details, please refer to the build report. |
|
Learn Build status updates of commit 71235d7:
|
| File | Status | Preview URL | Details |
|---|---|---|---|
| docs/develop/enable-nested-app-authentication-in-your-add-in.md | View | Details | |
| docs/outlook/faq-nested-app-auth-outlook-legacy-tokens.md | ✅Succeeded | View |
docs/develop/enable-nested-app-authentication-in-your-add-in.md
- Line 140, Column 1: [Warning: code-block-indented - See documentation]
Indented code blocks aren't allowed. Use a Markdown code block surrounded by triple backticks (```).
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.
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 |
| NAA supports both Microsoft Accounts and Microsoft Entra ID (work/school) identities. It doesn't support [Azure Active Directory B2C](/azure/active-directory-b2c/overview) for business-to-consumer identity management scenarios. For more information on NAA requirements, see [Nested app auth requirement set](/javascript/api/requirement-sets/common/nested-app-auth-requirement-sets). | ||
|
|
||
| ## Register your single-page application | ||
|
|
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.
@davidchesnut Do we need to add a note/line about needing another redirect to your actual domain (like a normal SPA app) for when on the web in WXP, given brk-multihub only covers the host applications and OWA which do brokering?
mattgeim
left a comment
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.
added one comment about redirect registration
|
Learn Build status updates of commit ede1278: ❌ Validation status: errorsPlease follow instructions here which may help to resolve issue.
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. |
|
Learn Build status updates of commit bd6f44c: ❌ Validation status: errorsPlease follow instructions here which may help to resolve issue.
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. |
|
Learn Build status updates of commit a3b8e39:
|
| File | Status | Preview URL | Details |
|---|---|---|---|
| docs/develop/enable-nested-app-authentication-in-your-add-in.md | View | Details | |
| docs/develop/enable-single-sign-on-in-an-office-add-in.md | ✅Succeeded | View | |
| docs/toc.yml | ✅Succeeded | View |
docs/develop/enable-nested-app-authentication-in-your-add-in.md
- Line 143, Column 1: [Warning: code-block-indented - See documentation]
Indented code blocks aren't allowed. Use a Markdown code block surrounded by triple backticks (```).
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.
PoliCheck Scan ReportThe following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. 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 |
PoliCheck Scan ReportThe following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. 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 2bfd808:
|
| File | Status | Preview URL | Details |
|---|---|---|---|
| docs/develop/enable-nested-app-authentication-in-your-add-in.md | View | Details | |
| docs/images/azure-portal-add-taskpane-redirect-uri.png | 💡Suggestion | View | Details |
| docs/images/azure-portal-create-app-reg-naa.png | 💡Suggestion | View | Details |
| docs/develop/enable-single-sign-on-in-an-office-add-in.md | ✅Succeeded | View | |
| docs/images/azure-portal-add-uri.png | ✅Succeeded | View | |
| docs/images/azure-portal-copy-client-id.png | ✅Succeeded | View | |
| docs/toc.yml | ✅Succeeded | View |
docs/develop/enable-nested-app-authentication-in-your-add-in.md
- Line 143, Column 1: [Warning: code-block-indented - See documentation]
Indented code blocks aren't allowed. Use a Markdown code block surrounded by triple backticks (```).
docs/images/azure-portal-add-taskpane-redirect-uri.png
- Line 0, Column 0: [Suggestion: image-name-incomplete - See documentation]
The image name contains an incomplete word, misspelled word, acronym, or abbreviation that is disallowed.
docs/images/azure-portal-create-app-reg-naa.png
- Line 0, Column 0: [Suggestion: image-name-incomplete - See documentation]
The image name contains an incomplete word, misspelled word, acronym, or abbreviation that is disallowed.
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.
PoliCheck Scan ReportThe following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. 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 ff994b0: ✅ Validation status: passed
For more details, please refer to the build report. |
PoliCheck Scan ReportThe following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. 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 582a8eb: ✅ Validation status: passed
For more details, please refer to the build report. |
PoliCheck Scan ReportThe following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. 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 838f68d: ✅ Validation status: passed
For more details, please refer to the build report. |
|
Learn Build status updates of commit 6457afe:
|
| File | Status | Preview URL | Details |
|---|---|---|---|
| docs/develop/enable-nested-app-authentication-in-your-add-in.md | View | Details | |
| docs/develop/enable-single-sign-on-in-an-office-add-in.md | View | Details | |
| docs/images/azure-portal-add-task-pane-redirect-uri.png | ✅Succeeded | View | |
| docs/images/azure-portal-add-uri.png | ✅Succeeded | View | |
| docs/images/azure-portal-copy-client-id.png | ✅Succeeded | View | |
| docs/images/azure-portal-create-application-registration-nested-app-authentication.png | ✅Succeeded | View | |
| docs/toc.yml | ✅Succeeded | View |
docs/develop/enable-nested-app-authentication-in-your-add-in.md
- Line 9, Column 1: [Warning: duplicate-h1s - See documentation]
H1 'Enable single sign-on in an Office Add-in with nested app authentication' is duplicated with other articles: 'develop/enable-nested-app-authentication-in-your-add-in.md(9,1), develop/enable-single-sign-on-in-an-office-add-in.md(9,1)'. First level headings must be unique within a docset.
docs/develop/enable-single-sign-on-in-an-office-add-in.md
- Line 9, Column 1: [Warning: duplicate-h1s - See documentation]
H1 'Enable single sign-on in an Office Add-in with nested app authentication' is duplicated with other articles: 'develop/enable-nested-app-authentication-in-your-add-in.md(9,1), develop/enable-single-sign-on-in-an-office-add-in.md(9,1)'. First level headings must be unique within a docset.
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.
PoliCheck Scan ReportThe following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. 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 |
PoliCheck Scan ReportThe following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. 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 21b92fe:
|
| File | Status | Preview URL | Details |
|---|---|---|---|
| docs/develop/enable-nested-app-authentication-in-your-add-in.md | View | Details | |
| docs/develop/enable-single-sign-on-in-an-office-add-in.md | View | Details | |
| docs/images/azure-portal-add-task-pane-redirect-uri.png | ✅Succeeded | View | |
| docs/images/azure-portal-add-uri.png | ✅Succeeded | View | |
| docs/images/azure-portal-copy-client-id.png | ✅Succeeded | View | |
| docs/images/azure-portal-create-application-registration-nested-app-authentication.png | ✅Succeeded | View | |
| docs/toc.yml | ✅Succeeded | View |
docs/develop/enable-nested-app-authentication-in-your-add-in.md
- Line 9, Column 1: [Warning: duplicate-h1s - See documentation]
H1 'Enable single sign-on in an Office Add-in with nested app authentication' is duplicated with other articles: 'develop/enable-nested-app-authentication-in-your-add-in.md(9,1), develop/enable-single-sign-on-in-an-office-add-in.md(9,1)'. First level headings must be unique within a docset.
docs/develop/enable-single-sign-on-in-an-office-add-in.md
- Line 9, Column 1: [Warning: duplicate-h1s - See documentation]
H1 'Enable single sign-on in an Office Add-in with nested app authentication' is duplicated with other articles: 'develop/enable-nested-app-authentication-in-your-add-in.md(9,1), develop/enable-single-sign-on-in-an-office-add-in.md(9,1)'. First level headings must be unique within a docset.
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.
PoliCheck Scan ReportThe following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. 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 113a856: ✅ Validation status: passed
For more details, please refer to the build report. |