Skip to content

Add 'Generate API Key' button for LLM providers in Settings panel#7

Merged
superhq-dev merged 3 commits intosuperhq-ai:mainfrom
AdarshKrishnan007:add-api-key-helper
Jul 16, 2025
Merged

Add 'Generate API Key' button for LLM providers in Settings panel#7
superhq-dev merged 3 commits intosuperhq-ai:mainfrom
AdarshKrishnan007:add-api-key-helper

Conversation

@AdarshKrishnan007
Copy link
Copy Markdown
Contributor

Summary

  • Added a 'Get Key' button next to API key input field in Settings.tsx
  • Uses a modular PROVIDER_URLS map to open the correct page
  • Opens link in new tab with fallback handling
  • Tooltip: "API key generation page"

✅ Satisfies all requirements from the issue.
📌 Onboarding.tsx is not present in the codebase, so change was only made in Settings.tsx.

Let me know if anything needs adjusting!

@superhq-dev
Copy link
Copy Markdown
Collaborator

Hi @AdarshKrishnan007, thank you for the PR. There are a couple of minor issues that need to be addressed before we can merge it:

  1. Please avoid committing package-lock files, especially when no new dependencies have been added. The changes in this file likely stem from a minor version bump related to Vite.
  2. There's a minor merge conflict that needs to be resolved.

Otherwise, the PR looks good 🚀

@superhq-dev
Copy link
Copy Markdown
Collaborator

Awesome! I have resolved the merge conflict and am merging it now.
Again, thanks a lot for the PR! really appreciate it! I hope more contributions will follow.

@superhq-dev superhq-dev merged commit 25875ef into superhq-ai:main Jul 16, 2025
@AdarshKrishnan007 AdarshKrishnan007 deleted the add-api-key-helper branch July 16, 2025 15:00
@AdarshKrishnan007
Copy link
Copy Markdown
Contributor Author

Thank you so much for merging the PR and guiding me through! 😊
I really enjoyed working on this issue and would love to contribute more. Let me know if there’s anything else I can help with!

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