-
Notifications
You must be signed in to change notification settings - Fork 68
[Outlook] Revamp Mailbox requirement set pages #2472
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
base: main
Are you sure you want to change the base?
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.
More information about PoliCheckInformation: PoliCheck | Severity Guidance | Term |
|
Learn Build status updates of commit 05cb7f4:
|
| File | Status | Preview URL | Details |
|---|---|---|---|
| docs/requirement-sets/outlook/outlook-requirement-set-1.1.md | View (>=common-js || >=outlook-js-1.1 || >=excel-js-1.1 || onenote-js-1.1 || >=powerpoint-js-1.1 || visio-js-1.1 || >=word-js-1.1) | Details | |
| docs/requirement-sets/outlook/outlook-requirement-set-1.10.md | View (>=common-js || >=outlook-js-1.1 || >=excel-js-1.1 || onenote-js-1.1 || >=powerpoint-js-1.1 || visio-js-1.1 || >=word-js-1.1) | Details | |
| docs/requirement-sets/outlook/outlook-requirement-set-1.11.md | View (>=common-js || >=outlook-js-1.1 || >=excel-js-1.1 || onenote-js-1.1 || >=powerpoint-js-1.1 || visio-js-1.1 || >=word-js-1.1) | Details | |
| docs/requirement-sets/outlook/outlook-requirement-set-1.12.md | View (>=common-js || >=outlook-js-1.1 || >=excel-js-1.1 || onenote-js-1.1 || >=powerpoint-js-1.1 || visio-js-1.1 || >=word-js-1.1) | Details | |
| docs/requirement-sets/outlook/outlook-requirement-set-1.13.md | View (>=common-js || >=outlook-js-1.1 || >=excel-js-1.1 || onenote-js-1.1 || >=powerpoint-js-1.1 || visio-js-1.1 || >=word-js-1.1) | Details | |
| docs/requirement-sets/outlook/outlook-requirement-set-1.14.md | View (>=common-js || >=outlook-js-1.1 || >=excel-js-1.1 || onenote-js-1.1 || >=powerpoint-js-1.1 || visio-js-1.1 || >=word-js-1.1) | Details | |
| docs/requirement-sets/outlook/outlook-requirement-set-1.15.md | View (>=common-js || >=outlook-js-1.1 || >=excel-js-1.1 || onenote-js-1.1 || >=powerpoint-js-1.1 || visio-js-1.1 || >=word-js-1.1) | Details | |
| docs/requirement-sets/outlook/outlook-requirement-set-1.2.md | View (>=common-js || >=outlook-js-1.1 || >=excel-js-1.1 || onenote-js-1.1 || >=powerpoint-js-1.1 || visio-js-1.1 || >=word-js-1.1) | Details | |
| docs/requirement-sets/outlook/outlook-requirement-set-1.3.md | View (>=common-js || >=outlook-js-1.1 || >=excel-js-1.1 || onenote-js-1.1 || >=powerpoint-js-1.1 || visio-js-1.1 || >=word-js-1.1) | Details | |
| docs/requirement-sets/outlook/outlook-requirement-set-1.4.md | View (>=common-js || >=outlook-js-1.1 || >=excel-js-1.1 || onenote-js-1.1 || >=powerpoint-js-1.1 || visio-js-1.1 || >=word-js-1.1) | Details | |
| docs/requirement-sets/outlook/outlook-requirement-set-1.5.md | View (>=common-js || >=outlook-js-1.1 || >=excel-js-1.1 || onenote-js-1.1 || >=powerpoint-js-1.1 || visio-js-1.1 || >=word-js-1.1) | Details | |
| docs/requirement-sets/outlook/outlook-requirement-set-1.6.md | View (>=common-js || >=outlook-js-1.1 || >=excel-js-1.1 || onenote-js-1.1 || >=powerpoint-js-1.1 || visio-js-1.1 || >=word-js-1.1) | Details | |
| docs/requirement-sets/outlook/outlook-requirement-set-1.7.md | View (>=common-js || >=outlook-js-1.1 || >=excel-js-1.1 || onenote-js-1.1 || >=powerpoint-js-1.1 || visio-js-1.1 || >=word-js-1.1) | Details | |
| docs/requirement-sets/outlook/outlook-requirement-set-1.8.md | View (>=common-js || >=outlook-js-1.1 || >=excel-js-1.1 || onenote-js-1.1 || >=powerpoint-js-1.1 || visio-js-1.1 || >=word-js-1.1) | Details | |
| docs/requirement-sets/outlook/outlook-requirement-set-1.9.md | View (>=common-js || >=outlook-js-1.1 || >=excel-js-1.1 || onenote-js-1.1 || >=powerpoint-js-1.1 || visio-js-1.1 || >=word-js-1.1) | Details | |
| .openpublishing.redirection.json | ✅Succeeded | View (>=common-js || >=outlook-js-1.1 || >=excel-js-1.1 || onenote-js-1.1 || >=powerpoint-js-1.1 || visio-js-1.1 || >=word-js-1.1) View |
|
| docs/docs-ref-autogen/common_preview/toc.yml | ✅Succeeded | View (common-js-preview || outlook-js-preview || excel-js-preview || onenote-js-1.1 || powerpoint-js-preview || word-js-preview) | |
| docs/docs-ref-autogen/common/toc.yml | ✅Succeeded | View (>=common-js || >=outlook-js-1.1 || >=excel-js-1.1 || onenote-js-1.1 || >=powerpoint-js-1.1 || visio-js-1.1 || >=word-js-1.1) View (common-js || <outlook-js-preview || <excel-js-preview || <powerpoint-js-preview || <word-js-preview) View (common-js-preview || outlook-js-preview || excel-js-preview || onenote-js-1.1 || powerpoint-js-preview || word-js-preview) |
|
| docs/docs-ref-autogen/excel_1_1/toc.yml | ✅Succeeded | View (excel-js-1.1) | |
| docs/docs-ref-autogen/excel_1_10/toc.yml | ✅Succeeded | View (excel-js-1.10) | |
| docs/docs-ref-autogen/excel_1_11/toc.yml | ✅Succeeded | View (excel-js-1.11) | |
| docs/docs-ref-autogen/excel_1_12/toc.yml | ✅Succeeded | View (excel-js-1.12) | |
| docs/docs-ref-autogen/excel_1_13/toc.yml | ✅Succeeded | View (excel-js-1.13) | |
| docs/docs-ref-autogen/excel_1_14/toc.yml | ✅Succeeded | View (excel-js-1.14) | |
| docs/docs-ref-autogen/excel_1_15/toc.yml | ✅Succeeded | View (excel-js-1.15) |
This comment lists only the first 25 files in the pull request.
docs/requirement-sets/outlook/outlook-requirement-set-1.1.md
- Line 0, Column 0: [Warning: filename-invalid-character - See documentation]
Only lowercase letters, numbers, and hyphens are allowed in filenames.
docs/requirement-sets/outlook/outlook-requirement-set-1.10.md
- Line 0, Column 0: [Warning: filename-invalid-character - See documentation]
Only lowercase letters, numbers, and hyphens are allowed in filenames.
docs/requirement-sets/outlook/outlook-requirement-set-1.11.md
- Line 0, Column 0: [Warning: filename-invalid-character - See documentation]
Only lowercase letters, numbers, and hyphens are allowed in filenames.
docs/requirement-sets/outlook/outlook-requirement-set-1.12.md
- Line 0, Column 0: [Warning: filename-invalid-character - See documentation]
Only lowercase letters, numbers, and hyphens are allowed in filenames.
docs/requirement-sets/outlook/outlook-requirement-set-1.13.md
- Line 0, Column 0: [Warning: filename-invalid-character - See documentation]
Only lowercase letters, numbers, and hyphens are allowed in filenames.
docs/requirement-sets/outlook/outlook-requirement-set-1.14.md
- Line 0, Column 0: [Warning: filename-invalid-character - See documentation]
Only lowercase letters, numbers, and hyphens are allowed in filenames.
docs/requirement-sets/outlook/outlook-requirement-set-1.15.md
- Line 0, Column 0: [Warning: filename-invalid-character - See documentation]
Only lowercase letters, numbers, and hyphens are allowed in filenames.
docs/requirement-sets/outlook/outlook-requirement-set-1.2.md
- Line 0, Column 0: [Warning: filename-invalid-character - See documentation]
Only lowercase letters, numbers, and hyphens are allowed in filenames.
docs/requirement-sets/outlook/outlook-requirement-set-1.3.md
- Line 0, Column 0: [Warning: filename-invalid-character - See documentation]
Only lowercase letters, numbers, and hyphens are allowed in filenames.
docs/requirement-sets/outlook/outlook-requirement-set-1.4.md
- Line 0, Column 0: [Warning: filename-invalid-character - See documentation]
Only lowercase letters, numbers, and hyphens are allowed in filenames.
docs/requirement-sets/outlook/outlook-requirement-set-1.5.md
- Line 0, Column 0: [Warning: filename-invalid-character - See documentation]
Only lowercase letters, numbers, and hyphens are allowed in filenames.
docs/requirement-sets/outlook/outlook-requirement-set-1.6.md
- Line 0, Column 0: [Warning: filename-invalid-character - See documentation]
Only lowercase letters, numbers, and hyphens are allowed in filenames.
docs/requirement-sets/outlook/outlook-requirement-set-1.7.md
- Line 0, Column 0: [Warning: filename-invalid-character - See documentation]
Only lowercase letters, numbers, and hyphens are allowed in filenames.
docs/requirement-sets/outlook/outlook-requirement-set-1.8.md
- Line 0, Column 0: [Warning: filename-invalid-character - See documentation]
Only lowercase letters, numbers, and hyphens are allowed in filenames.
docs/requirement-sets/outlook/outlook-requirement-set-1.9.md
- Line 0, Column 0: [Warning: filename-invalid-character - See documentation]
Only lowercase letters, numbers, and hyphens are allowed in filenames.
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.
AlexJerabek
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.
Looks good to me. I think this will be much easier to maintain going forward.
#policheck-false-positive since this is documenting code. |
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.
More information about PoliCheckInformation: PoliCheck | Severity Guidance | Term |
|
Learn Build status updates of commit 0e9cf7d: ✅ Validation status: passed
This comment lists only the first 25 files in the pull request. |
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.
More information about PoliCheckInformation: PoliCheck | Severity Guidance | Term |
|
Learn Build status updates of commit 85e131a: ✅ Validation status: passed
This comment lists only the first 25 files in the pull request. |
|
@samantharamon This is a great reorganization - thx for doing this. The only concern I have is regarding item. Do you have another location in mind to capture this matrix of the modes that support the various members (example)? In the descriptions of the various item flavors, we direct users to the Object Model i.e., office.context.mailbox.item (e.g., see outlook package). |
Where applicable, Object Model references would link to the Office.Item page since it already lists the different Outlook modes with links to the specific interface pages, e.g., Office.AppointmentCompose. We may need to update the information on the |
filterUnionAdditionstofalsein all Outlook config files.includeEnumstotruein all Outlook config files.whats-newtool. This includes updates to WXP includes files.versioned-d.ts-toolsto0.8.0.Separate PRs will be created to update the requirement set links in other repositories.