search

OfficeDev / office-js-docs

[ARCHIVED] Microsoft Office Add-ins API Reference Documentation
https://docs.microsoft.com/javascript/api/overview/office
398 stars 247 forks source link

Requirement Set documentation is not up to date concerning hosts #1463

Closed ndeeH closed 5 years ago

ndeeH commented 6 years ago

When looking in the methods for Example Document.goToByIdAsync there is PowerPoint Online available. In the Requirement Sets there is only PowerPoint noted. Please could you update all sets to represent the current status for each host.

Rick-Kirkham commented 6 years ago

@weird-dreams Could you clarify the problem? When I look at the reference topic https://dev.office.com/reference/add-ins/shared/document.gotobyidasync, it says that the method is not in any Requirement Set.

ndeeH commented 6 years ago

Here is another example.

Requirement Set: image so Selection schould not be available for PowerPoint Online

The Documentation for each method says it is available for PowerPoint Online: image

image

When we use the requirement set it seems that the method perspective wins - the add in is also visible in PowerPoint Online.

ElizabethSamuel-MSFT commented 5 years ago

@weird-dreams: Thanks for pointing this out. The issues that you raised have already been addressed - see Document, Host and platform availability, Common API requirement sets. However, we'll do another pass to ensure that we have consistent support details across the documentation.

ElizabethSamuel-MSFT commented 5 years ago

@weird-dreams: We've updated the details per these PRs (https://github.com/OfficeDev/office-js-docs-reference/pull/247, https://github.com/OfficeDev/office-js-docs-pr/pull/568) and the changes should go live soon. If you do notice any other discrepancies, please let us know. Thanks.

ndeeH commented 5 years ago

@ElizabethSamuel-MSFT Thank you!