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

Consolidate credentials documentation #3164

Closed
datajoely opened this issue Oct 11, 2023 · 2 comments
Closed

Consolidate credentials documentation #3164

datajoely opened this issue Oct 11, 2023 · 2 comments
Labels
Issue: Feature Request New feature or improvement to existing feature

Comments

@datajoely
Copy link
Contributor

datajoely commented Oct 11, 2023

Description

There is lots of useful credential related information in the docs, but it's not actually all in the main credentials page.

Env vars and hooks are our recommended pattern for advanced users - we don't make this clear in the main credentials page.

  • Env vars are linked in this page, but it's not actually very visible.
  • After context created hooks are our recommended pattern for advanced users and this is not linked at all in the docs.

I propose we move all the examples and details of the above patterns to the main credentials page and link to that from the advanced config and hook examples pages respectively.

@datajoely datajoely added the Issue: Feature Request New feature or improvement to existing feature label Oct 11, 2023
@astrojuanlu
Copy link
Member

Duplicate (or at least related) to #2669 ?

@datajoely
Copy link
Contributor Author

Dupe will close

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Issue: Feature Request New feature or improvement to existing feature
Projects
None yet
Development

No branches or pull requests

2 participants