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

Assistant: ensure that ASSISTANT_API_KEY is a real synonym for OPEN_AI_KEY #1237

Merged
merged 1 commit into from
Oct 3, 2024

Conversation

jordigh
Copy link
Contributor

@jordigh jordigh commented Oct 1, 2024

Context

This was flagged as inconsistent in a community post.

https://community.getgrist.com/t/does-the-ai-formula-assist-also-works-with-the-desktop-mac-app/5677/6

Proposed solution

Define the apiKey variable earlier in the call stack.

Has this been tested?

  • 👍 yes, I added tests to the test suite
  • 💭 no, because this PR is a draft and still needs work
  • 🙅 no, because this is not relevant here
  • 🙋 no, because I need help

@jordigh jordigh force-pushed the jordigh/openai-key branch from 20a05a0 to 858bd53 Compare October 1, 2024 20:40
@jordigh jordigh force-pushed the jordigh/openai-key branch from 858bd53 to 50e6bd3 Compare October 2, 2024 11:56
@jordigh jordigh merged commit 35a429d into main Oct 3, 2024
11 checks passed
@jordigh jordigh deleted the jordigh/openai-key branch October 3, 2024 13:55
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