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

setting google API key to blank #1369

Conversation

neutralvision
Copy link

@neutralvision neutralvision commented Apr 14, 2023

Background

since google API key is set by default, AutoGPT will try to call invalid API. removing API key string so that this code is triggered:
https://github.com/Torantulino/Auto-GPT/blob/master/scripts/commands.py#L63

Changes

setting google API key to blank in env template.

Test Plan

no

PR Quality Checklist

  • My pull request is atomic and focuses on a single change.
  • I have thoroughly tested my changes with multiple different prompts.
  • I have considered potential risks and mitigations for my changes.
  • I have documented my changes clearly and comprehensively.
  • I have not snuck in any "extra" small tweaks changes

@hdkiller
Copy link
Contributor

hdkiller commented Apr 14, 2023

This PR which recently merged was referencing to the default value. I suppose that should be removed when this merged.

@github-actions github-actions bot added the conflicts Automatically applied to PRs with merge conflicts label Apr 19, 2023
@github-actions
Copy link
Contributor

This pull request has conflicts with the base branch, please resolve those so we can evaluate the pull request.

@p-i-
Copy link
Contributor

p-i- commented May 5, 2023

This is a mass message from the AutoGPT core team.
Our apologies for the ongoing delay in processing PRs.
This is because we are re-architecting the AutoGPT core!

For more details (and for infor on joining our Discord), please refer to:
https://github.com/Significant-Gravitas/Auto-GPT/wiki/Architecting

@kinance
Copy link
Contributor

kinance commented May 22, 2023

By default, the google API key is commented out. It shouldn't have the invalid API error. I think this PR can be closed. Please reopen if mistaken.

@kinance kinance closed this May 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
conflicts Automatically applied to PRs with merge conflicts
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants