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

Update .env-template #250

Merged
merged 1 commit into from
Jun 1, 2023
Merged

Update .env-template #250

merged 1 commit into from
Jun 1, 2023

Conversation

tardigrde
Copy link
Contributor

Rename env var name to OPENAI_API_KEY.
This was needed for me to run the docker-compose (prod) setup.

Few questions:

  • what about API_KEY? is it used anywhere else than inside the docker containers?
  • what about EMBEDDINGS_KEY? is it needed? also see that it is used in containes (only?)

@vercel
Copy link

vercel bot commented May 26, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
docs-gpt ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 26, 2023 7:02am

@tardigrde
Copy link
Contributor Author

tardigrde commented May 27, 2023

Well I realized, that in the https://github.com/arc53/DocsGPT#quickstart it says:
Create .env file in your root directory and set your OPENAI_API_KEY with your openai api key

For me this was confusing as I expected that i'd just have to rename .env-template to .env, set my key and I'm good to go. I did not realize that in the quickstart a new .env has to be setup, for which we do not have a template. Is that right?

I set the PR as draft, I guess there is no need to merge it now. What would be nice, is a more consoldiated way of treating env vars. I can help you with that, if you have an idea how you would like to have it.

EDIT: I cannot set the state to Draft

@tardigrde tardigrde changed the title Update .env-template WIP: Update .env-template May 27, 2023
@tardigrde tardigrde changed the title WIP: Update .env-template Update .env-template May 27, 2023
Copy link
Contributor

@dartpain dartpain left a comment

Choose a reason for hiding this comment

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

Sorry for taking so long, thank you so much must have missed that

@dartpain dartpain merged commit bbf55ca into arc53:main Jun 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants