Skip to content

[excel] (Custom Functions) Reorganize CF Auth article for better engagement#5798

Merged
AlexJerabek merged 3 commits into
mainfrom
AlexJ-CFAuth
Jul 1, 2026
Merged

[excel] (Custom Functions) Reorganize CF Auth article for better engagement#5798
AlexJerabek merged 3 commits into
mainfrom
AlexJ-CFAuth

Conversation

@AlexJerabek

Copy link
Copy Markdown
Contributor

This PR reorganizes the CF Auth article for engagement. It pulls the workflow bulletpoints to the beginning and groups the Dialog and Storage samples more cohesively.

@learn-build-service-prod

Copy link
Copy Markdown
Contributor

Learn Build status updates of commit 9b0e245:

✅ Validation status: passed

File Status Preview URL Details
docs/excel/custom-functions-authentication.md ✅Succeeded View

For more details, please refer to the build report.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR reorganizes the Excel custom functions authentication article to surface the recommended auth workflow earlier and improve sample grouping/readability for JavaScript-only runtime scenarios.

Changes:

  • Rewrote the introduction and added an explicit “Authentication workflow” section near the top.
  • Added a “Try it with a sample” section earlier, then regrouped the Dialog and OfficeRuntime.storage guidance.
  • Updated the article metadata (description, ms.date) and normalized a code fence language tag.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread docs/excel/custom-functions-authentication.md Outdated
Comment thread docs/excel/custom-functions-authentication.md Outdated
@learn-build-service-prod

Copy link
Copy Markdown
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.

@davidchesnut davidchesnut left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks great!

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@learn-build-service-prod

Copy link
Copy Markdown
Contributor

Learn Build status updates of commit 323009a:

✅ Validation status: passed

File Status Preview URL Details
docs/excel/custom-functions-authentication.md ✅Succeeded View

For more details, please refer to the build report.

@learn-build-service-prod

Copy link
Copy Markdown
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.

@AlexJerabek AlexJerabek merged commit f196c7d into main Jul 1, 2026
3 checks passed
@AlexJerabek AlexJerabek deleted the AlexJ-CFAuth branch July 1, 2026 22:49
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.

3 participants