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

Minor changes to collections docs #612

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

Conversation

christad92
Copy link
Contributor

Short Description

A one or two-sentence description of what this PR does.

Closes #

Details

Add any more details that may be relevant to the reviewer. How did you approach
these docs? Are there any parts you struggled with, or that need particularly
careful review?

AI Usage

Please disclose how you've used AI in this work (it's cool, we just want to
know!):

  • Code generation (copilot but not intellisense)
  • Learning or fact checking
  • Strategy / design
  • Optimisation / refactoring
  • Translation / spellchecking / doc gen
  • Other
  • I have not used AI

You can read more details in our
Responsible AI Policy


When running in the CLI, a Personal Access Token can be used to get access to
the collection (generated from the app at /profile/tokens).

See the [Collections](/documentation/build/collections) Platform Docs to learn
more about Collections.
Learn more about collections and common use cases [here](/documentation/build/collections).
Copy link
Contributor

Choose a reason for hiding this comment

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

Hey, in formal technical documentation, I don't think it's appropriate to use here as link text. What is here? Where does the link go? It's needlessly mysterious: better to clearly signpost what we're linking to.

It's usually better to use the page title (even if it might drift out of date over time). But since this page is also called Collections it's probably better to maybe say Collections Documentation or even Main Collections Documentation

Copy link
Contributor Author

Choose a reason for hiding this comment

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

uhmm.. will review and push again. Thanks

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

Successfully merging this pull request may close these issues.

2 participants