-
Notifications
You must be signed in to change notification settings - Fork 293
enhance: ask user for OpenAI key and store it in the cred store #396
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
Conversation
pkg/gptscript/gptscript.go
Outdated
return nil, err | ||
} | ||
|
||
oAIClient, err := openai.NewClient(cliCfg, opts.CredentialContext, opts.OpenAI, openai.Options{ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit:
oAIClient, err := openai.NewClient(cliCfg, opts.CredentialContext, opts.OpenAI, openai.Options{ | |
oaiClient, err := openai.NewClient(cliCfg, opts.CredentialContext, opts.OpenAI, openai.Options{ |
d9190b5
to
145a039
Compare
I made some pretty substantial changes to this PR:
|
We will need to make sure this works with @tylerslaton's UI, once he has prompting support |
d1f3e8a
to
49e83fe
Compare
Okay, this is finally ready again. Darren wanted me to add support for the remote client to prompt for credentials as well, so that is the main thing I introduced. I also had to make some changes to accommodate the changes in the prompt server. |
62807c0
to
2b377fc
Compare
Signed-off-by: Grant Linville <[email protected]>
Signed-off-by: Grant Linville <[email protected]>
10a2287
to
f8cbdd5
Compare
Signed-off-by: Grant Linville <[email protected]>
No description provided.