Skip to content

Add Addin API doc #8840

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

Merged
merged 5 commits into from
Apr 4, 2023
Merged

Add Addin API doc #8840

merged 5 commits into from
Apr 4, 2023

Conversation

RongqiZ
Copy link
Contributor

@RongqiZ RongqiZ commented Mar 24, 2023

Category

  • Content fix
  • New article

Related issues

None

What's in this Pull Request?

Added one doc about Add-in APIs. They could help to have a clear view and better management of the Add-ins in tenant level, including these:

  • Get available Add-ins in sites
  • Get Add-in permissions in site collections
  • Get tenant ACS service principals
  • Get Add-in principals in site collections
  • Uninstall Add-ins
  • Get uninstall Add-in job status

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 41d5370:

✅ Validation status: passed

File Status Preview URL Details
docs/apis/addin-restful-apis.md ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 2113807:

💡 Validation status: suggestions

File Status Preview URL Details
docs/toc.yml 💡Suggestion View Details
docs/apis/addin-management-apis.md ✅Succeeded View

docs/toc.yml

  • Line 2134, Column 11: [Suggestion: link-broken - See documentation] Link '/apis/addin-management-apis.md' points to a page that doesn't exist. Check the path or URL and update the link.

For more details, please refer to the build report.

Note: Links can become broken if there are changes on the target sites. If you think the validation result is a false alarm, please use SiteHelp with option 'Content authoring and publishing -> False alarm on broken link validation' to 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:

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 452a213:

✅ Validation status: passed

File Status Preview URL Details
docs/apis/addin-management-apis.md ✅Succeeded View
docs/toc.yml ✅Succeeded View

For more details, please refer to the build report.

For any questions, please:

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 357a247:

✅ Validation status: passed

File Status Preview URL Details
docs/apis/addin-management-apis.md ✅Succeeded View
docs/toc.yml ✅Succeeded View

For more details, please refer to the build report.

For any questions, please:

@VesaJuvonen VesaJuvonen self-assigned this Mar 28, 2023

| Name | Type | Description |
|---------|--------|-----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------|
| feature | string | The feature name of the permissions, it should be one of these:<br/> <li> Taxonomy <br/> <li> Social <br/> <li> ProjectServer <br/> <li> Search <br/> <li> BcsConnection <br/> <li> Content |
Copy link
Collaborator

Choose a reason for hiding this comment

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

This isn't a valid bullet list. Remove the <br /> tags and surround all <li> with the correct <ul> element.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Updated, thanks!

@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 labels Mar 29, 2023
@ghost ghost removed the Needs: Author Feedback Awaiting response from the original poster of the issue. Marked as stale if no activity for 7 days. label Apr 3, 2023
@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 6db58f0:

✅ Validation status: passed

File Status Preview URL Details
docs/apis/addin-management-apis.md ✅Succeeded View
docs/toc.yml ✅Succeeded View

For more details, please refer to the build report.

For any questions, please:

@andrewconnell
Copy link
Collaborator

LGTM... because this is a new doc, I need to defer to MSFT (@VesaJuvonen) to merge it into the main branch.

@andrewconnell andrewconnell added pr:ready-to-merge Ready to merge PR and removed pr:do-not-merge Pull request not ready to be merged labels Apr 4, 2023
@VesaJuvonen VesaJuvonen merged commit 051c5de into SharePoint:main Apr 4, 2023
@VesaJuvonen
Copy link
Contributor

thx @RongqiZ for this - will get published today as part of other docs. Great work on this! 👏👏👏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr:ready-to-merge Ready to merge PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants