[GSSoC'26] fix: resolve hearing principals by email#1122
Conversation
|
@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. |
|
Hi @vipul674, thanks for contributing to Nyay Setu! 🎉 I have automatically:
Our workflows will now analyze your changes to classify:
Tip Ensure your PR description references the issue it resolves (e.g. Happy coding! 🚀 |
|
@viru0909-dev — All CI checks are green. This PR is ready for review and merge. Merge-Ready PR ReportVerdictMERGE_READY (all GitHub Actions CI green; Vercel deploy is external authorization blocker only) Issue and Assignment
Files Changed
Testing & VerificationAll 17 frontend tests pass. Backend tests pass. NLP tests pass. Build-and-push passes. CI Status
Known Limitations
GSSoC 2026 ComplianceThis contribution was prepared with AI assistance and manually reviewed before submission. Closes #1080 |
|
Approved. Both backend and frontend tests pass cleanly. Good work. Resolve merge conflict it seem both PR has same merge conflict. |
Summary
Files Changed
backend/nyaysetu-backend/src/main/java/com/nyaysetu/backend/controller/HearingController.java: resolves the current user id throughAuthService.findByEmail(authentication.getName()).backend/nyaysetu-backend/src/test/java/com/nyaysetu/backend/controller/HearingControllerTest.java: adds focused join/leave regression tests using email principals.Testing Performed
mvn -Dtest=HearingControllerTest testmvn -DskipTests packageKnown Limitations
mvn testis blocked in this Java 25 environment by an existing Mockito/Byte Buddy incompatibility in unrelated tests (Java 25 (69) is not supported by the current version of Byte Buddy).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 #1080