Skip to content

feat(ui): improve settings modal and header interactions#508

Open
sohika-sharma20 wants to merge 2 commits into
param20h:devfrom
sohika-sharma20:feat-settings-modal
Open

feat(ui): improve settings modal and header interactions#508
sohika-sharma20 wants to merge 2 commits into
param20h:devfrom
sohika-sharma20:feat-settings-modal

Conversation

@sohika-sharma20
Copy link
Copy Markdown

@sohika-sharma20 sohika-sharma20 commented Jun 7, 2026

🔗 Related Issue

Closes #98

📝 What does this PR do?

Adds a settings modal in the header and improves header interactions.
Updates chat panel behavior and API integration for the settings functionality.

🗂️ Type of Change

  • 🐛 Bug fix
  • ✨ New feature
  • 🔧 Refactor / code cleanup
  • 📝 Documentation update
  • 🎨 UI / styling change
  • ⚙️ CI / tooling / config change
  • 🧪 Tests

🧪 How was this tested?

  • Ran the backend locally (uvicorn app.main:app --reload)
  • Ran the frontend locally (npm run dev inside frontend/)
  • Tested the affected API endpoints manually
  • Added / updated tests

📸 Screenshots (if UI change)

Screenshot 2026-06-07 130458

⚠️ Anything to flag for reviewers?

GitHub reports merge conflicts with the latest upstream dev branch because the same files were modified upstream after this change was implemented.

✅ Self-Review Checklist

  • My branch is based on dev, not main
  • I have not added any secrets / API keys
  • I have not modified main branch or any HuggingFace deployment config
  • My code follows the existing style
  • I have updated relevant docs / comments if needed

@sohika-sharma20 sohika-sharma20 requested a review from param20h as a code owner June 7, 2026 08:50
@param20h
Copy link
Copy Markdown
Owner

param20h commented Jun 7, 2026

chcks failing @sohika-sharma20

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.

✨ feat(ui): Implement a Settings Modal for LLM configuration

2 participants