-
Notifications
You must be signed in to change notification settings - Fork 1k
Fix typos in formatting-syntax-reference and form-customizer docs #10474
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
Conversation
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 1a12873:
|
| File | Status | Preview URL | Details |
|---|---|---|---|
| docs/declarative-customization/formatting-syntax-reference.md | View | Details | |
| docs/spfx/extensions/get-started/building-form-customizer.md | ✅Succeeded | View |
docs/declarative-customization/formatting-syntax-reference.md
- Line 384, Column 223: [Warning: disallowed-html-tag - See documentation]
HTML tag 'ItemID' isn't allowed. Replace it with approved Markdown or escape the brackets if the content is a placeholder.
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 294ab4f: ✅ Validation status: passed
For more details, please refer to the build report. |
@microsoft-github-policy-service agree company="Microsoft" |
|
Can't review until CLA is completed. @Ashlesha-MSFT please review & complete these instructiosn: #10474 (comment) This is required for all submissions in this repo. |
|
@andrewconnell, |
- `id` is a property, so it should be formatted as code - fixed other grammatical issues throughout doc - update timestamp
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 |
andrewconnell
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.
In the future, please separate two very different changes in different PRs
|
Learn Build status updates of commit 7d49a47: ✅ Validation status: passed
For more details, please refer to the build report. |
Category
Related issues
embedaction supported for embedding a list item display form(DispForm.aspx) ? #10299embedaction supported in document libraries? #10296What's in this Pull Request?