Skip to content

Update form customizer getting-started doc wording#10949

Open
HarminderSethi wants to merge 9 commits into
SharePoint:mainfrom
HarminderSethi:main
Open

Update form customizer getting-started doc wording#10949
HarminderSethi wants to merge 9 commits into
SharePoint:mainfrom
HarminderSethi:main

Conversation

@HarminderSethi

Copy link
Copy Markdown
Contributor

Category

  • Content fix
  • New article

What's in this Pull Request?

Update form customizer getting-started doc wording

@learn-build-service-prod

Copy link
Copy Markdown
Contributor

Learn Build status updates of commit eed207d:

✅ Validation status: passed

File Status Preview URL Details
docs/spfx/extensions/get-started/building-form-customizer.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.

@andrewconnell andrewconnell self-assigned this Jul 15, 2026
@andrewconnell andrewconnell added the pr:in-review Actively reviewing pull request label Jul 15, 2026
- ContentType.**EditFormClientSideComponentProperties** - optional configuration details

1. Compile your code and host the compiled files from the local machine by running this command:
2. Compile your code and host the compiled files from the local machine by running this command:

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

don't renumber ordered bullets, all bullets should be 1. and let the rendering engine number them at build time. please revert this.

This will start your default browser and load the page defined in **serve.json** file.

1. Accept the loading of debug manifests by selecting **Load debug scripts** when prompted.
3. Accept the loading of debug manifests by selecting **Load debug scripts** when prompted.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

don't renumber ordered bullets, all bullets should be 1. and let the rendering engine number them at build time. please revert this.

```

1. Update the **_onSave** methods in the the **HelloWorldFormCustomizer** class as follows.
2. Update the **_onSave** methods in the **HelloWorldFormCustomizer** class as follows.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

don't renumber ordered bullets, all bullets should be 1. and let the rendering engine number them at build time. please revert this.

```

1. Add new method **_createItem** to the **HelloWorldFormCustomizer** class.
3. Add new method **_createItem** to the **HelloWorldFormCustomizer** class.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

don't renumber ordered bullets, all bullets should be 1. and let the rendering engine number them at build time. please revert this.

Comment thread docs/spfx/extensions/get-started/building-form-customizer.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 Jul 15, 2026
@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 1bbccdb:

✅ Validation status: passed

File Status Preview URL Details
docs/spfx/extensions/get-started/building-form-customizer.md ✅Succeeded View

For more details, please refer to the build report.

@HarminderSethi

Copy link
Copy Markdown
Contributor Author

@andrewconnell I have reverted the order numbers. Not able to understand how it got changed 🤨

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

Labels

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants