fix: update CSP and security meta tags#1148
Conversation
|
@PranavAgarkar07 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 @PranavAgarkar07, 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 This PR is ready for review — CI passes, all labels applied (including gssoc:approved). Could you take a look and merge when convenient? Thanks! |
viru0909-dev
left a comment
There was a problem hiding this comment.
Requires Changes. Please address the following issues:
- PR modifies UI components but no screenshot or video is attached in the description.
Frontend tests failed:\n\n ❯ src/pages/litigant/DocumentGeneratePage.test.jsx (4 tests | 2 failed) 1376ms ⎯⎯⎯⎯⎯⎯⎯ Failed Tests 2 ⎯⎯⎯⎯⎯⎯⎯ FAIL src/pages/litigant/DocumentGeneratePage.test.jsx > DocumentGeneratePage > copies generated content and calls DOCX export actions FAIL src/pages/litigant/DocumentGeneratePage.test.jsx > DocumentGeneratePage > downloads PDF and handles filename and blob flow Test Files 1 failed | 4 passed (5) Tests 2 failed | 14 passed (16)\n
viru0909-dev
left a comment
There was a problem hiding this comment.
Review Summary
This PR has been reviewed and requires changes before it can be approved. The issues below must be resolved.
Issues to Resolve
| Result | Check | Notes |
|---|---|---|
| FAIL | Description | Description is missing: a Type of Change section; a Checklist section with checkboxes. |
| FAIL | Checklist | No checklist found in the description. |
| FAIL | UI Screenshot | This PR modifies UI files but no screenshot or video is attached. Add before/after screenshots or a screen recording showing the change. |
Passing Checks
| Result | Check | Notes |
|---|---|---|
| PASS | Title | Title format is acceptable. |
| PASS | Issue Link | Issue is linked with a closing keyword. |
| PASS | Merge Conflicts | Conflict status is not yet computed by GitHub — skipping. |
| PASS | Branch Freshness | Branch was last updated 2.5 days ago. |
| PASS | CI / Tests | No CI checks are configured or results are not yet available. |
| PASS | Code Quality | No code quality issues detected in the diff. |
To proceed: push the required fixes to fix/1001-csp-security-meta-tags and the PR will be re-evaluated on the next review run.
5114404 to
1888179
Compare
Description
Changes
Type of Change
Checklist
Screenshots
N/A — This PR only modifies HTML meta tags and CSP directives in
index.html. There are no visible UI changes. The Google Fonts loading fix has no visual diff, and the PWA meta tags are only relevant when the site is added to the iOS home screen.Related Issue
Closes #1001