-
Notifications
You must be signed in to change notification settings - Fork 18
Update collections docs for latest cli #730
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
base: main
Are you sure you want to change the base?
Conversation
|
@mtuchi Thanks for the comments. I've cleaned that section up - it's just supposed to be a better summary but it was not an improvement 😅 Should be better now |
adaptors/collections.md
Outdated
| The rest of this guide only applies when using older CLI versions, prior to | ||
| 1.21.0. | ||
|
|
||
| Remember that a Collection must be created from the Admin page before it can be |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remember that a Collection must be created from the project settings page before it can be used
adaptors/collections.md
Outdated
| But if you're unable to update your CLI, you can still configure collections | ||
| manually with the [Older CLI guide](#older-cli). | ||
| Remember that a Collection must be created from the Admin page before it can be |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry @josephjclark hate to be picky 😬 but, any user can create collections from project settings page now
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ah great catch, thanks!
| #### For a workflow | ||
| If you're using `workflow.json`, set the token and endpoint on | ||
| `workflow.credentials`: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wrong key, this should be workflow.configuration
|
@josephjclark just two tiny issue, then this will be ready to merge |
Update collections overview to match latest CLI updates
AI Usage
Please disclose how you've used AI in this work (it's cool, we just want to
know!):
You can read more details in our
Responsible AI Policy