Skip to content

fix: multiple bug fixes#82

Open
Xenon010101 wants to merge 2 commits into
Payal-Dhokane:mainfrom
Xenon010101:fix/various-bugs
Open

fix: multiple bug fixes#82
Xenon010101 wants to merge 2 commits into
Payal-Dhokane:mainfrom
Xenon010101:fix/various-bugs

Conversation

@Xenon010101
Copy link
Copy Markdown

Summary of Fixes

  • Fixed: Chat feature was dead code (never imported/rendered) — wired into the UI
  • Fixed: README referenced OpenAI but code uses Groq — updated docs
  • Fixed: Removed unused imports (OAuth2Component, duplicate render_insight_card)

- Removed unused OAuth2Component import from src/auth.py
- Removed redundant render_insight_card imports from app.py
- Wired up previously dead chat.py component as 'Chat with Data' step in navigation
- Updated step indicator to include Chat step
- Changed prerequisites to mention Groq API key only
- Updated .env example to only show GROQ_API_KEY
- Fixed error table entry from OpenAI to GROQ_API_KEY
- Updated Tech Stack to list Groq instead of OpenAI/Groq
- Updated setup instructions to point to console.groq.com
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.

1 participant