Skip to content

Update graph and guidance docs to heft#10561

Merged
andrewconnell merged 12 commits intoSharePoint:mainfrom
Ashlesha-MSFT:update-graph-and-guidance-docs-heft
Jan 15, 2026
Merged

Update graph and guidance docs to heft#10561
andrewconnell merged 12 commits intoSharePoint:mainfrom
Ashlesha-MSFT:update-graph-and-guidance-docs-heft

Conversation

@Ashlesha-MSFT
Copy link
Copy Markdown
Contributor

@Ashlesha-MSFT Ashlesha-MSFT commented Jan 1, 2026

Category

  • Content fix

Related issues

What's in this Pull Request?

  • Update using-microsoft-graph-apis.md:

    • Remove migration WIP banner
    • Modernize Graph API calls from callback to Promise-based pattern
    • Add proper error handling with .catch()
    • Improve TypeScript typing and null safety
    • Update commands: gulp to heft (build/package-solution)
    • Add null coalescing operator and null checks
    • Fix code formatting with consistent indentation
  • Update simplify-adding-web-parts-with-preconfigured-entries.md:

    • Update ms.date to 01/01/2026
    • Migrate commands from gulp to heft
    • Update heft start command in all examples
  • Update graph-helloworld-webpart-on-page.png"

@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.

@learn-build-service-prod
Copy link
Copy Markdown
Contributor

Learn Build status updates of commit dfb2529:

✅ Validation status: passed

File Status Preview URL Details
docs/images/graph-helloworld-webpart-on-page.png ✅Succeeded View
docs/spfx/web-parts/get-started/add-jqueryui-accordion-to-web-part.md ✅Succeeded View
docs/spfx/web-parts/get-started/using-microsoft-graph-apis.md ✅Succeeded View
docs/spfx/web-parts/guidance/simplify-adding-web-parts-with-preconfigured-entries.md ✅Succeeded View

For more details, please refer to the build report.

@Ashlesha-MSFT
Copy link
Copy Markdown
Contributor Author

@andrewconnell ,
I noticed that some changes for this documentation weren't synced with docs/spfx/web-parts/guidance/simplify-adding-web-parts-with-preconfigured-entries.md from my local branch, so I've included those updates in this PR as well to ensure consistency across the documentation.

Going forward, I'll double-check PRs after creation to ensure all related changes are properly included before submission.

@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.

@learn-build-service-prod
Copy link
Copy Markdown
Contributor

@learn-build-service-prod
Copy link
Copy Markdown
Contributor

@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.

@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.

@learn-build-service-prod
Copy link
Copy Markdown
Contributor

@andrewconnell andrewconnell added the pr:in-review Actively reviewing pull request label Jan 7, 2026
@andrewconnell andrewconnell self-assigned this Jan 7, 2026
@andrewconnell andrewconnell self-requested a review January 7, 2026 11:13
Comment thread docs/images/graph-helloworld-webpart-on-page.png
Comment thread docs/spfx/web-parts/get-started/build-web-part-microsoft-graph-toolkit.md Outdated
@andrewconnell andrewconnell added Needs: Author Feedback Awaiting response from the original poster of the issue. Marked as stale if no activity for 7 days. pr:do-not-merge Pull request not ready to be merged and removed pr:in-review Actively reviewing pull request labels Jan 7, 2026
@andrewconnell
Copy link
Copy Markdown
Collaborator

Thanks for the submission @Ashlesha-MSFT ... few changes requested before we can merge this.

As its width does not match documentation
@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.

@learn-build-service-prod
Copy link
Copy Markdown
Contributor

Learn Build status updates of commit 411845f:

⚠️ Validation status: warnings

File Status Preview URL Details
docs/spfx/web-parts/get-started/using-microsoft-graph-apis.md ⚠️Warning View Details
docs/images/graph-helloworld-webpart-on-page.png ✅Succeeded n/a (file deleted or renamed)
docs/spfx/web-parts/get-started/add-jqueryui-accordion-to-web-part.md ✅Succeeded View
docs/spfx/web-parts/get-started/build-web-part-microsoft-graph-toolkit.md ✅Succeeded View
docs/spfx/web-parts/get-started/provision-sp-assets-from-package.md ✅Succeeded View
docs/spfx/web-parts/get-started/using-web-part-as-ms-teams-tab.md ✅Succeeded View
docs/spfx/web-parts/guidance/simplify-adding-web-parts-with-preconfigured-entries.md ✅Succeeded View

docs/spfx/web-parts/get-started/using-microsoft-graph-apis.md

  • Line 230, Column 5: [Warning: file-not-found - See documentation] Invalid file link: '../../../images/graph-helloworld-webpart-on-page.png'.

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
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.

@learn-build-service-prod
Copy link
Copy Markdown
Contributor

@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.

@learn-build-service-prod
Copy link
Copy Markdown
Contributor

@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.

@learn-build-service-prod
Copy link
Copy Markdown
Contributor

@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.

@learn-build-service-prod
Copy link
Copy Markdown
Contributor

@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.

@learn-build-service-prod
Copy link
Copy Markdown
Contributor

@andrewconnell andrewconnell added pr:in-review Actively reviewing pull request and removed Needs: Author Feedback Awaiting response from the original poster of the issue. Marked as stale if no activity for 7 days. pr:do-not-merge Pull request not ready to be merged labels Jan 15, 2026
@andrewconnell andrewconnell added pr:ready-to-merge Ready to merge PR and removed pr:in-review Actively reviewing pull request labels Jan 15, 2026
@andrewconnell andrewconnell merged commit 9866421 into SharePoint:main Jan 15, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment