Skip to content

docs: fix doc store code snippet in README.md#1389

Merged
anna-charlotte merged 4 commits intomainfrom
docs-readme-doc-store
Apr 17, 2023
Merged

docs: fix doc store code snippet in README.md#1389
anna-charlotte merged 4 commits intomainfrom
docs-readme-doc-store

Conversation

@anna-charlotte
Copy link
Copy Markdown
Contributor

Fix doc store code example in the README.md (using jina ai cloud example instead of s3 bucket)

Signed-off-by: anna-charlotte <[email protected]>
@anna-charlotte anna-charlotte changed the title docs: fix doc store code snippet docs: fix doc store code snippet in README.md Apr 17, 2023
@anna-charlotte anna-charlotte marked this pull request as ready for review April 17, 2023 10:13
README.md Outdated


# first create a personal access token here: https://cloud.jina.ai/settings/tokens
os.environ['JINA_AUTH_TOKEN'] = 'YOUR_PAT'
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

will this work even if the auth token is YOU_PAT ?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

ah no, you're right, won't work.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

will change to the on-disk example

@anna-charlotte anna-charlotte requested a review from alexcg1 April 17, 2023 10:45
Co-authored-by: Alex Cureton-Griffiths <[email protected]>
Signed-off-by: Charlotte Gerhaher <[email protected]>
@github-actions
Copy link
Copy Markdown

📝 Docs are deployed on https://ft-docs-readme-doc-store--jina-docs.netlify.app 🎉

@anna-charlotte anna-charlotte requested a review from alexcg1 April 17, 2023 11:39
Copy link
Copy Markdown
Contributor

@alexcg1 alexcg1 left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@anna-charlotte anna-charlotte merged commit ce3fb6e into main Apr 17, 2023
@anna-charlotte anna-charlotte deleted the docs-readme-doc-store branch April 17, 2023 11:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants