Skip to content

fix: Replace API key with obvious dummy#7175

Open
nicksalvemini-edb wants to merge 2 commits into
developfrom
api-key-fix
Open

fix: Replace API key with obvious dummy#7175
nicksalvemini-edb wants to merge 2 commits into
developfrom
api-key-fix

Conversation

@nicksalvemini-edb
Copy link
Copy Markdown
Contributor

@nicksalvemini-edb nicksalvemini-edb commented May 22, 2026

Replace API key with obvious dummy

@nicksalvemini-edb nicksalvemini-edb requested a review from a team as a code owner May 22, 2026 20:10
@nicksalvemini-edb
Copy link
Copy Markdown
Contributor Author

This one is good to be merged whenever 👍

@irmabenitezzapata-creator irmabenitezzapata-creator added oncall deploy Add this label to a PR and it will automatically be deployed to netlify labels May 25, 2026
'gemini',
'{"model": "gemini-2.5-flash"}'::JSONB,
'{"api_key": "AIzaSyBXnKr4yJG2lMhyvza81q3-WFuP7Tk3r8z"}'::JSONB
'{"api_key": "AIza..."}'::JSONB
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
'{"api_key": "AIza..."}'::JSONB
'{"api_key": "AIzaSyXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX"}'::JSONB

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@nicksalvemini-edb, I left a suggestion on how to make it clearer to the user that they need to enter the API value by using a placeholder. Let me know if this works for you, and I'll go ahead and merge the PR. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

deploy Add this label to a PR and it will automatically be deployed to netlify oncall

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants