Skip to content
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

[All Hosts] (manifest) update tech background articles for unified manifest #4834

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

Rick-Kirkham
Copy link
Contributor

No description provided.

Copy link
Contributor

Learn Build status updates of commit 0e45d55:

❌ Validation status: errors

Please follow instructions here which may help to resolve issue.

File Status Preview URL Details
❌Error Details

  • [Error: CannotMergeCommit] Cannot merge commit 0e45d553cb8baac7798c849f0223296894a55cde in branch unified-manifest-tech of repository https://github.com/OfficeDev/office-js-docs-pr into branch main (commit ebe5bd2bdc6702910d0f006e451ecdbb67f76d10). Please follow this documentation: https://help.github.com/articles/resolving-a-merge-conflict-using-the-command-line/ to use git.exe to resolve you content conflicts locally and then push to remote.

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.

For any questions, please:

Copy link
Contributor

Learn Build status updates of commit 2ced768:

⚠️ Validation status: warnings

File Status Preview URL Details
docs/design/contextual-tabs.md ⚠️Warning View Details
docs/develop/dialog-api-in-office-add-ins.md ⚠️Warning View Details
docs/develop/specify-api-requirements-runtime.md ⚠️Warning View Details
docs/develop/specify-office-hosts-and-api-requirements-unified.md ⚠️Warning View Details
docs/develop/specify-office-hosts-and-api-requirements.md ⚠️Warning View Details
docs/toc.yml ✅Succeeded View

docs/design/contextual-tabs.md

  • Line 30, Column 144: [Warning: bookmark-not-found - See documentation] Cannot find bookmark '#runtime-checks-for-method-and-requirement-set-support' in 'develop/specify-office-hosts-and-api-requirements.md'.

docs/develop/dialog-api-in-office-add-ins.md

  • Line 333, Column 300: [Warning: bookmark-not-found - See documentation] Cannot find bookmark '#runtime-checks-for-method-and-requirement-set-support' in 'develop/specify-office-hosts-and-api-requirements.md'.

docs/develop/specify-api-requirements-runtime.md

  • Line 5, Column 10: [Warning: ms-date-invalid - See documentation] Value out of range for 'ms.date': '11/04/2024'. If specified, must be a date in format M/D/YYYY, no more than 30 days from today. If you don't specify ms.date, the date will be updated automatically.
  • Line 63, Column 326: [Warning: bookmark-not-found - See documentation] Cannot find bookmark '#use-the-latest-office-javascript-api-library' in 'develop/specify-api-requirements-runtime.md'.

docs/develop/specify-office-hosts-and-api-requirements-unified.md

  • Line 5, Column 10: [Warning: ms-date-invalid - See documentation] Value out of range for 'ms.date': '11/04/2024'. If specified, must be a date in format M/D/YYYY, no more than 30 days from today. If you don't specify ms.date, the date will be updated automatically.
  • Line 126, Column 97: [Suggestion: learn-site-link-broken - See documentation] Link '/develop/requirements-property-unified-manifest.md#extensionsrequirements' points to a learn site page that doesn't exist. Check the path or URL and update the link.
  • Line 160, Column 64: [Suggestion: learn-site-link-broken - See documentation] Link '/develop/requirements-property-unified-manifest.md#filter-features' points to a learn site page that doesn't exist. Check the path or URL and update the link.
  • Line 166, Column 3: [Suggestion: learn-site-link-broken - See documentation] Link '/develop/requirements-property-unified-manifest.md' points to a learn site page that doesn't exist. Check the path or URL and update the link.

docs/develop/specify-office-hosts-and-api-requirements.md

  • Line 5, Column 10: [Warning: ms-date-invalid - See documentation] Value out of range for 'ms.date': '11/04/2024'. If specified, must be a date in format M/D/YYYY, no more than 30 days from today. If you don't specify ms.date, the date will be updated automatically.

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.

For any questions, please:

Copy link
Contributor

Learn Build status updates of commit 334b032:

✅ Validation status: passed

File Status Preview URL Details
docs/design/contextual-tabs.md ✅Succeeded View
docs/develop/dialog-api-in-office-add-ins.md ✅Succeeded View
docs/develop/specify-api-requirements-runtime.md ✅Succeeded View
docs/develop/specify-office-hosts-and-api-requirements-unified.md ✅Succeeded View
docs/develop/specify-office-hosts-and-api-requirements.md ✅Succeeded View
docs/toc.yml ✅Succeeded View

For more details, please refer to the build report.

For any questions, please:

Copy link
Contributor

@samantharamon samantharamon left a comment

Choose a reason for hiding this comment

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

Looks good. Added minor suggestions for your consideration and a question.

docs/develop/dialog-api-in-office-add-ins.md Outdated Show resolved Hide resolved
docs/develop/specify-api-requirements-runtime.md Outdated Show resolved Hide resolved
docs/develop/specify-api-requirements-runtime.md Outdated Show resolved Hide resolved
docs/develop/specify-api-requirements-runtime.md Outdated Show resolved Hide resolved
docs/develop/specify-api-requirements-runtime.md Outdated Show resolved Hide resolved
Copy link
Contributor

Learn Build status updates of commit 68d688c:

✅ Validation status: passed

File Status Preview URL Details
docs/design/contextual-tabs.md ✅Succeeded View
docs/develop/dialog-api-in-office-add-ins.md ✅Succeeded View
docs/develop/specify-api-requirements-runtime.md ✅Succeeded View
docs/develop/specify-office-hosts-and-api-requirements-unified.md ✅Succeeded View
docs/develop/specify-office-hosts-and-api-requirements.md ✅Succeeded View
docs/toc.yml ✅Succeeded View

For more details, please refer to the build report.

For any questions, please:

| Name | Office client applications | Available add-in types |
|:--------------|:-----------------------------------------------|:-----------------------|
| document | Word on the web, Windows, Mac, iPad | Task pane |
| mail | Outlook on the web, Windows ([new](https://support.microsoft.com/office/656bb8d9-5a60-49b2-a98b-ba7822bc7627) and classic), Mac, Android, iOS | Mail |
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good catch. It isn't.

Copy link
Contributor

Learn Build status updates of commit 4b2bad3:

✅ Validation status: passed

File Status Preview URL Details
docs/design/contextual-tabs.md ✅Succeeded View
docs/develop/dialog-api-in-office-add-ins.md ✅Succeeded View
docs/develop/specify-api-requirements-runtime.md ✅Succeeded View
docs/develop/specify-office-hosts-and-api-requirements-unified.md ✅Succeeded View
docs/develop/specify-office-hosts-and-api-requirements.md ✅Succeeded View
docs/toc.yml ✅Succeeded View

For more details, please refer to the build report.

For any questions, please:

Copy link
Contributor

Learn Build status updates of commit 27bbc3b:

✅ Validation status: passed

File Status Preview URL Details
docs/design/contextual-tabs.md ✅Succeeded View
docs/develop/dialog-api-in-office-add-ins.md ✅Succeeded View
docs/develop/specify-api-requirements-runtime.md ✅Succeeded View
docs/develop/specify-office-hosts-and-api-requirements-unified.md ✅Succeeded View
docs/develop/specify-office-hosts-and-api-requirements.md ✅Succeeded View
docs/resources/resources-glossary.md ✅Succeeded View
docs/toc.yml ✅Succeeded View

For more details, please refer to the build report.

For any questions, please:

docs/toc.yml Outdated Show resolved Hide resolved
Copy link
Contributor

Learn Build status updates of commit 1e61ab0:

✅ Validation status: passed

File Status Preview URL Details
docs/design/contextual-tabs.md ✅Succeeded View
docs/develop/dialog-api-in-office-add-ins.md ✅Succeeded View
docs/develop/specify-api-requirements-runtime.md ✅Succeeded View
docs/develop/specify-office-hosts-and-api-requirements-unified.md ✅Succeeded View
docs/develop/specify-office-hosts-and-api-requirements.md ✅Succeeded View
docs/resources/resources-glossary.md ✅Succeeded View

For more details, please refer to the build report.

For any questions, please:

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

Successfully merging this pull request may close these issues.

2 participants