Skip to content

Create docs for new StorageProvider and activity-related properties #2087

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

Draft
wants to merge 3 commits into
base: docs
Choose a base branch
from

Conversation

amittere
Copy link
Contributor

Adding documentation for new StorageProvider and Activity related properties

Copy link
Contributor

@amittere : Thanks for your contribution! The author(s) and reviewer(s) have been notified to review your proposed change.

@alvinashcraft
Copy link
Contributor

Thanks for adding these, @amittere. You'll also need to add them to the toc.yml file in the properties folder so they appear in the table of contents on the left side of the Learn pages. Also, whichever folder they're added to in the TOC, you should also add them to the corresponding Topic/Description table in the *-bumper.md file for that folder. Those act as indexes for each folder to link to the individual property pages.

@jwmsft may have addition advice/suggestions since he owns this area of the docs. It's the first time I've worked with someone on adding new content here.

@jwmsft
Copy link
Contributor

jwmsft commented Jul 29, 2025

Property docs source files aren't organized into folders, only the bumpers and TOC entries are grouped. The existing StoragePrivider props are in the Core bumper, which seems to be a catch-all for things that aren't in another group.

  • It might make sense to move all the StorageProvider properties out of Core into a new TOC/bumper group of their own.
  • ContentUri probably makes sense to add to Core.
  • The 3 Activity props could go in Core or in a new Activities group.

Let me know how you'd like to handle these, and I can help with the TOC updates if you'd like.

@jwmsft jwmsft marked this pull request as draft August 12, 2025 16:33
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.

3 participants