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 vars #94

Merged
merged 1 commit into from
Apr 29, 2024
Merged

Update Env vars #94

merged 1 commit into from
Apr 29, 2024

Conversation

knno
Copy link
Contributor

@knno knno commented Apr 16, 2024

Hi @danenania
I would like to say thanks for working on this project. and I hope my contribution is useful. Here is what I did:

  • Changed OPENAI_ENDPOINT to OPENAI_API_BASE
  • CLI: Added PLANDEX_API_HOST to set the server host for development
  • DEV: Added PLANDEX_OUT_DIR to set where to export "plandex.exe" file.
  • SERVER: Fixed ports to use 8080 by default instead of 8088
  • Docs: Updated and added a ENVIRONMENT_VARS.md file

Also added *.exe to the gitignore.

Please let me know if there are any changes you'd like.

- Changed OPENAI_ENDPOINT to OPENAI_API_BASE
- CLI: Added PLANDEX_API_HOST to set the server host for development
- DEV: Added PLANDEX_OUT_DIR to set where to export "plandex.exe" file.
- SERVER: Fixed ports to use 8080 by default instead of 8088
- Docs: Updated and added a ENVIRONMENT_VARS.md file
@danenania
Copy link
Contributor

danenania commented Apr 17, 2024

Thank you @knno! I'll have time to look this over soon. One quick comment for now: though I know OPENAI_API_BASE is is more standard/canonical than OPENAI_ENDPOINT and I'm happy with making it the new default, I'd like to keep legacy support for OPENAI_ENDPOINT as well for people who have already started using that. It can just be supported quietly though--no need to mention it in readme/guides.

@danenania danenania merged commit 2503f80 into plandex-ai:main Apr 29, 2024
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.

None yet

2 participants