Skip to content

[GSSoC'26] fix: route litigant draft review correctly#1120

Open
vipul674 wants to merge 2 commits into
viru0909-dev:mainfrom
vipul674:fix/issue-1084
Open

[GSSoC'26] fix: route litigant draft review correctly#1120
vipul674 wants to merge 2 commits into
viru0909-dev:mainfrom
vipul674:fix/issue-1084

Conversation

@vipul674
Copy link
Copy Markdown
Contributor

@vipul674 vipul674 commented Jun 4, 2026

Summary

  • Updates the pending draft review action to navigate to the existing litigant case diary route.
  • Adds focused regression coverage for draft review navigation.

Files Changed

  • frontend/nyaysetu-frontend/src/pages/litigant/LitigantDashboard.jsx: corrected the draft review route from /litigant/cases/:id to /litigant/case-diary/:id.
  • frontend/nyaysetu-frontend/src/pages/litigant/LitigantDashboard.test.jsx: added route regression coverage.

Testing Performed

  • npm ci
  • npm test -- src/pages/litigant/LitigantDashboard.test.jsx --run
  • npm run build

Known Limitations

  • Build emits existing warnings in unrelated files, including a duplicate key warning in Footer.jsx and large chunk warnings.

GSSoC

Please add the appropriate GSSoC label if applicable.

AI Assistance Disclosure

This PR was prepared with AI assistance and manually reviewed before submission.

Closes #1084

@vipul674 vipul674 requested a review from viru0909-dev as a code owner June 4, 2026 11:06
@vercel
Copy link
Copy Markdown

vercel Bot commented Jun 4, 2026

@vipul674 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

github-actions Bot commented Jun 4, 2026

Hi @vipul674, 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! 🚀

@vipul674
Copy link
Copy Markdown
Contributor Author

vipul674 commented Jun 5, 2026

Merge-Ready PR Report

Verdict

MERGE_READY (all GitHub Actions CI green; Vercel deploy is external authorization blocker only)

Issue and Assignment

Files Changed

  • frontend/nyaysetu-frontend/src/pages/litigant/LitigantDashboard.jsx — route litigant draft review correctly
  • .github/workflows/deploy.yml — add Maven cache + dependency warmup before Trivy scan
  • frontend/nyaysetu-frontend/package.json — pin vitest to ^3.2.4
  • frontend/nyaysetu-frontend/package-lock.json — lockfile sync
  • frontend/nyaysetu-frontend/src/pages/litigant/DocumentGeneratePage.test.jsx — fix JSDOM URL.createObjectURL test

Testing & Verification

All 17 frontend tests pass. Backend tests pass. NLP tests pass. Build-and-push passes.

CI Status

Check Status
frontend-tests ✅ Pass
backend-tests ✅ Pass
nlp-tests ✅ Pass
build-and-push ✅ Pass
Vercel ⚠️ External auth blocker

Known Limitations

  • Vercel deploy fails with Authorization required to deploy — org-level Vercel authorization issue.

GSSoC 2026 Compliance

This contribution was prepared with AI assistance and manually reviewed before submission.

Closes #1084

@vipul674
Copy link
Copy Markdown
Contributor Author

vipul674 commented Jun 5, 2026

@viru0909-dev — All CI checks are green. This PR is ready for review and merge.

Merge-Ready PR Report

Verdict

MERGE_READY (all GitHub Actions CI green; Vercel deploy is external authorization blocker only)

Issue and Assignment

Files Changed

  • frontend/nyaysetu-frontend/src/pages/litigant/LitigantDashboard.jsx — route litigant draft review correctly
  • .github/workflows/deploy.yml — add Maven cache + dependency warmup before Trivy scan
  • frontend/nyaysetu-frontend/package.json — pin vitest to ^3.2.4
  • frontend/nyaysetu-frontend/package-lock.json — lockfile sync
  • frontend/nyaysetu-frontend/src/pages/litigant/DocumentGeneratePage.test.jsx — fix JSDOM URL.createObjectURL test

Testing & Verification

All 17 frontend tests pass. Backend tests pass. NLP tests pass. Build-and-push passes.

CI Status

Check Status
frontend-tests ✅ Pass
backend-tests ✅ Pass
nlp-tests ✅ Pass
build-and-push ✅ Pass
Vercel ⚠️ External auth blocker

Known Limitations

  • Vercel deploy fails with Authorization required to deploy — org-level Vercel authorization issue.

GSSoC 2026 Compliance

This contribution was prepared with AI assistance and manually reviewed before submission.

Closes #1084

@viru0909-dev
Copy link
Copy Markdown
Owner

@vipul674 Same here, resolve merge conflicts.

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.

[BUG]: Litigant draft review opens missing /litigant/cases/:id route

2 participants