Skip to content

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

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

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

Conversation

@sohika-sharma20

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

Copy link
Copy Markdown

🔗 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

param20h commented Jun 7, 2026

Copy link
Copy Markdown
Owner

chcks failing @sohika-sharma20

@sohika-sharma20

Copy link
Copy Markdown
Author

chcks failing @sohika-sharma20

@param20h I pushed a new commit to retrigger CI. The workflows are awaiting approval.

param20h
param20h previously approved these changes Jun 11, 2026
param20h
param20h previously approved these changes Jun 11, 2026
@param20h

Copy link
Copy Markdown
Owner

solve

@sohika-sharma20

Copy link
Copy Markdown
Author

solve

Fixed the TypeScript build errors and updated the PR. Could you please re-review and approve the workflows? Thanks!

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