Skip to content

fix: duplicate config vars; gather exception handling; PUT body in FIR API - #1655

Open
saurabhhhcodes wants to merge 1 commit into
viru0909-dev:mainfrom
saurabhhhcodes:fix/nyay-batch-1
Open

fix: duplicate config vars; gather exception handling; PUT body in FIR API#1655
saurabhhhcodes wants to merge 1 commit into
viru0909-dev:mainfrom
saurabhhhcodes:fix/nyay-batch-1

Conversation

@saurabhhhcodes

Copy link
Copy Markdown

Bug 1: config.py had duplicate definitions of GROQ_MODEL_FAST and GEMINI_MODEL (lines 23-24 and 31-32). Removed the redundant second definitions.\n\nBug 2: research.py used asyncio.gather(return_exceptions=False) causing all tasks to be cancelled when one sub-question failed. Changed to return_exceptions=True with graceful fallback responses.\n\nBug 3: api.js updateFirStatus sent status and reviewNotes as query params with a null body for a PUT request. Changed to send data in the request body.

@vercel

vercel Bot commented Jul 25, 2026

Copy link
Copy Markdown

Someone is attempting to deploy a commit to the CodeBlooded's projects Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions

Copy link
Copy Markdown
Contributor

Hi @saurabhhhcodes, thanks for contributing to Nyay Setu! 🎉

I have automatically:

  • 👤 Assigned this PR to you.
  • 🏷️ Applied the gssoc:approved label.

Our workflows will now analyze your changes to classify:

  • 📈 PR Difficulty: level:*
  • 🧩 PR Type: type:*
  • 🌟 PR Quality: quality:*

Tip

Ensure your PR description references the issue it resolves (e.g. Closes #123). This allows the bot to inherit any additional labels from that issue!

Happy coding! 🚀

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant