Skip to content

Conversation

@davidchesnut
Copy link
Member

  • Updates for NAA GA.
  • Adds code for login hint required for Word, Excel, and PowerPoint on the web.

@davidchesnut davidchesnut self-assigned this Oct 14, 2025
@davidchesnut davidchesnut added the Status: Do Not Merge Do Not Merge label Oct 14, 2025
@learn-build-service-prod
Copy link
Contributor

PoliCheck Scan Report

The 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 found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 14a15ae:

✅ Validation status: passed

File Status Preview URL Details
docs/develop/enable-nested-app-authentication-in-your-add-in.md ✅Succeeded View

For more details, please refer to the build report.

@learn-build-service-prod
Copy link
Contributor

PoliCheck Scan Report

The 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 found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit d868b84:

✅ Validation status: passed

File Status Preview URL Details
docs/develop/enable-nested-app-authentication-in-your-add-in.md ✅Succeeded View
docs/outlook/faq-nested-app-auth-outlook-legacy-tokens.md ✅Succeeded View

For more details, please refer to the build report.

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 71235d7:

⚠️ Validation status: warnings

File Status Preview URL Details
docs/develop/enable-nested-app-authentication-in-your-add-in.md ⚠️Warning 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.

@learn-build-service-prod
Copy link
Contributor

PoliCheck Scan Report

The 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 found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

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

Copy link
Contributor

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?

Copy link
Contributor

@mattgeim mattgeim left a 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-service-prod
Copy link
Contributor

Learn Build status updates of commit ede1278:

❌ Validation status: errors

Please follow instructions here which may help to resolve issue.

File Status Preview URL Details
❌Error Details

  • [Error: CannotMergeCommit] Cannot merge commit ede12787452be169aa916ee0cd57030846742b2b in branch davech-naa-ga of repository https://github.com/OfficeDev/office-js-docs-pr into branch main (commit b99b5694cddb9d18c3ecc458e3f8d212fa509b73). Please follow this documentation: https://help.github.com/articles/resolving-a-merge-conflict-using-the-command-line/ to use git.exe to resolve you content conflicts locally and then push to remote.

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-service-prod
Copy link
Contributor

Learn Build status updates of commit bd6f44c:

❌ Validation status: errors

Please follow instructions here which may help to resolve issue.

File Status Preview URL Details
❌Error Details

  • [Error: CannotMergeCommit] Cannot merge commit bd6f44c00c41ef3491a6f33b07b5c55a53bed122 in branch davech-naa-ga of repository https://github.com/OfficeDev/office-js-docs-pr into branch main (commit b99b5694cddb9d18c3ecc458e3f8d212fa509b73). Please follow this documentation: https://help.github.com/articles/resolving-a-merge-conflict-using-the-command-line/ to use git.exe to resolve you content conflicts locally and then push to remote.

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-service-prod
Copy link
Contributor

Learn Build status updates of commit a3b8e39:

⚠️ Validation status: warnings

File Status Preview URL Details
docs/develop/enable-nested-app-authentication-in-your-add-in.md ⚠️Warning 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.

@learn-build-service-prod
Copy link
Contributor

PoliCheck Scan Report

The 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 found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

@learn-build-service-prod
Copy link
Contributor

PoliCheck Scan Report

The 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 found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 2bfd808:

⚠️ Validation status: warnings

File Status Preview URL Details
docs/develop/enable-nested-app-authentication-in-your-add-in.md ⚠️Warning 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.

@learn-build-service-prod
Copy link
Contributor

PoliCheck Scan Report

The 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 found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

@learn-build-service-prod
Copy link
Contributor

@learn-build-service-prod
Copy link
Contributor

PoliCheck Scan Report

The 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 found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

@learn-build-service-prod
Copy link
Contributor

@learn-build-service-prod
Copy link
Contributor

PoliCheck Scan Report

The 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 found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

@learn-build-service-prod
Copy link
Contributor

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 6457afe:

⚠️ Validation status: warnings

File Status Preview URL Details
docs/develop/enable-nested-app-authentication-in-your-add-in.md ⚠️Warning View Details
docs/develop/enable-single-sign-on-in-an-office-add-in.md ⚠️Warning 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.

@learn-build-service-prod
Copy link
Contributor

PoliCheck Scan Report

The 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 found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

@learn-build-service-prod
Copy link
Contributor

PoliCheck Scan Report

The 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 found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 21b92fe:

⚠️ Validation status: warnings

File Status Preview URL Details
docs/develop/enable-nested-app-authentication-in-your-add-in.md ⚠️Warning View Details
docs/develop/enable-single-sign-on-in-an-office-add-in.md ⚠️Warning 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.

@learn-build-service-prod
Copy link
Contributor

PoliCheck Scan Report

The 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 found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 113a856:

✅ Validation status: passed

File Status Preview URL Details
docs/develop/enable-nested-app-authentication-in-your-add-in.md ✅Succeeded View
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

For more details, please refer to the build report.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants