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

Make it deployable to Vercel #4

Merged
merged 6 commits into from
May 3, 2024

Conversation

jaredpalmer
Copy link
Contributor

@jaredpalmer jaredpalmer commented May 2, 2024

  • Makes the app more easily deployed by having assistantId be controlled by .env (process.env.OPENAI_ASSISTANT_ID)
  • Update readme to tell people to ensure file search is turned on when they create an their assistant
  • Add Vercel Deploy button to the readme

README.md Outdated
### 2. Set your [OpenAI API key](https://platform.openai.com/api-keys)
```shell
export OPENAI_API_KEY="sk_..."
### 2. Set your [OpenAI API key](https://platform.openai.com/api-keys) and [OpenAI Assistant ID](https://platform.openai.com/account/assistants)
Copy link

Choose a reason for hiding this comment

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

I get 404 when trying to navigate to https://platform.openai.com/account/assistant

README.md Outdated

```env
OPENAI_API_KEY="sk-123..."
OPENAI_ASSISTANT_ID="123..."
Copy link

Choose a reason for hiding this comment

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

to be more explicit I would change to
OPENAI_ASSISTANT_ID="asst_..."

@ibigio
Copy link
Collaborator

ibigio commented May 3, 2024

Thanks for this, totally makes sense! Just want to make sure the assistant setup flow still works with the new env var setup – might require an app restart instead of auto-updating on file save which is not as ideal, but going to poke around to see what I can get to work.

@ibigio
Copy link
Collaborator

ibigio commented May 3, 2024

Think I figured out a way that keeps the setup flow the same, but allows the env var to be set as well for Vercel deployment!

@ibigio ibigio merged commit 1fa3266 into openai:main May 3, 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

3 participants