prod: Main to Prod Merge by kushalmahawar2005 · Pull Request #20 · safe-cam/Shield-X-Web

kushalmahawar2005 · 2025-09-24T04:27:24Z

Main to prod

* chore: added labeler * fix: fix minor issues * chore: testing jobs * chore: updated workflows

* chore: added labeler * fix: fix minor issues * chore: updated ci jobs

* chore: added labeler * fix: fix minor issues * feat: #1 Auth Working and complete signup and login Funtionallity * fix : Fix Bugs or error * Fix : Env Variable Hided * Fix * init * chore: update workflows * chore: update workflows --------- Co-authored-by: kushal <kushalmahawar71@gmail.com>

* chore: added labeler * fix: fix minor issues * feat: #1 Auth Working and complete signup and login Funtionallity * fix : Fix Bugs or error * Fix : Env Variable Hided * Fix * init * chore: update workflows * chore: update workflows * chore: fix ci jobs * chore: Trigger CI re-run --------- Co-authored-by: kushal <kushalmahawar71@gmail.com>

* feat: #1 Auth Working and complete signup and login Funtionallity * fix : Fix Bugs or error * Fix : Env Variable Hided * Fix * init * chore: fix file name * chore: Trigger CI re-run * fix: updated package json * fix: added eslint * chore(web): run eslint on src and add minimal config * fix : eslint --------- Co-authored-by: Mohit Kumhar <mohitmolela@gmail.com>

vercel · 2025-09-24T04:27:31Z

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Preview	Comments	Updated (UTC)
shield-x-web-new	Ready	Preview	Comment	Sep 25, 2025 6:48pm
shield-x-web-slr4	Ready	Preview	Comment	Sep 25, 2025 6:48pm

1 Skipped Deployment

Project	Deployment	Preview	Comments	Updated (UTC)
shield-x-web	Ignored	Preview		Sep 25, 2025 6:48pm

Copilot

Pull Request Overview

This PR merges the main branch to the production branch, introducing a comprehensive web application implementation for ShieldX alongside server-side authentication capabilities.

Complete React web application (ShieldX-Web) with authentication, UI screens, and responsive design
Express.js server with MongoDB integration for user authentication
Updated CI/CD workflows to support the new web application structure

Reviewed Changes

Copilot reviewed 37 out of 56 changed files in this pull request and generated 4 comments.

Show a summary per file

File	Description
src/pages/Signup.jsx	File removed from original location
src/pages/Login.jsx	File removed from original location
ShieldX-Web/*	Complete new React web application with pages, components, and styling
server/*	New Express.js backend with authentication routes and MongoDB integration
.github/workflows/*	Updated CI/CD workflows to build and test the new web application

Comments suppressed due to low confidence (1)

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

Copilot · 2025-09-24T05:54:19Z

+
+
+        <div style={{ textAlign: 'center', marginTop: 12 }}>
+          <button className="link" type="button" onClick={() => navigate('/Signup')}>Don't have an account? <span style={{ color: 'red' }}>Sign Up</span></button>


The navigation route '/Signup' uses uppercase 'S' but the route is defined as '/signup' (lowercase) in App.jsx. This will cause navigation to fail.

Suggested change

<button className="link" type="button" onClick={() => navigate('/Signup')}>Don't have an account? <span style={{ color: 'red' }}>Sign Up</span></button>

<button className="link" type="button" onClick={() => navigate('/signup')}>Don't have an account? <span style={{ color: 'red' }}>Sign Up</span></button>

Copilot · 2025-09-24T05:54:20Z

+- POST /api/signup { name, email, password }
+- POST /api/login { email, password }


The API endpoint documentation is inconsistent with the actual implementation. The server uses '/api/register' not '/api/signup', and expects 'username' field, not 'email' field for both endpoints.

Suggested change

- POST /api/signup { name, email, password }

- POST /api/login { email, password }

- POST /api/register { name, username, password }

- POST /api/login { username, password }

Copilot · 2025-09-24T05:54:20Z

+  <head>
+    <meta charset="utf-8" />
+    <meta name="viewport" content="width=device-width, initial-scale=1.0" />
+    <title>SafeCam Converted Web</title>


The page title 'SafeCam Converted Web' doesn't match the application name 'ShieldX' used throughout the codebase.

Suggested change

<title>SafeCam Converted Web</title>

<title>ShieldX</title>

Copilot · 2025-09-24T05:54:20Z

@@ -0,0 +1,22 @@
+{
+  "name": "safe-cam-converted-web",


The package name 'safe-cam-converted-web' doesn't match the application name 'ShieldX' used throughout the codebase.

Suggested change

"name": "safe-cam-converted-web",

"name": "shieldx",

* chore: test * chore: fixes

* chore: test (#10) * chore: added ci jobs (#13) * chore: test (#9) * chore: added labeler * fix: fix minor issues * chore: testing jobs * chore: updated workflows * chore: updated wordflows (#11) * chore: added labeler * fix: fix minor issues * chore: testing jobs * chore: updated workflows * chore: updated ci jobs (#15) * chore: added labeler * fix: fix minor issues * chore: updated ci jobs * chore: update workflows (#17) * chore: added labeler * fix: fix minor issues * feat: #1 Auth Working and complete signup and login Funtionallity * fix : Fix Bugs or error * Fix : Env Variable Hided * Fix * init * chore: update workflows * chore: update workflows --------- Co-authored-by: kushal <kushalmahawar71@gmail.com> * chore: updated ci jobs (#18) * chore: added labeler * fix: fix minor issues * feat: #1 Auth Working and complete signup and login Funtionallity * fix : Fix Bugs or error * Fix : Env Variable Hided * Fix * init * chore: update workflows * chore: update workflows * chore: fix ci jobs * chore: Trigger CI re-run --------- Co-authored-by: kushal <kushalmahawar71@gmail.com> * Kushal (#19) * feat: #1 Auth Working and complete signup and login Funtionallity * fix : Fix Bugs or error * Fix : Env Variable Hided * Fix * init * chore: fix file name * chore: Trigger CI re-run * fix: updated package json * fix: added eslint * chore(web): run eslint on src and add minimal config * fix : eslint --------- Co-authored-by: Mohit Kumhar <mohitmolela@gmail.com> * chore: test (#10) * chore: fix authMiddlewere case-sensitive error * fix: conflicts * feat: Profile Section With All user detail Added * feat: Location Page Updated * fix: fix conflicts 2 --------- Co-authored-by: Mohit Kumhar <mohitmolela@gmail.com>

Copilot

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Copilot

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Copilot

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

* chore: test (#10) * chore: added ci jobs (#13) * chore: test (#9) * chore: added labeler * fix: fix minor issues * chore: testing jobs * chore: updated workflows * chore: updated wordflows (#11) * chore: added labeler * fix: fix minor issues * chore: testing jobs * chore: updated workflows * chore: updated ci jobs (#15) * chore: added labeler * fix: fix minor issues * chore: updated ci jobs * chore: update workflows (#17) * chore: added labeler * fix: fix minor issues * feat: #1 Auth Working and complete signup and login Funtionallity * fix : Fix Bugs or error * Fix : Env Variable Hided * Fix * init * chore: update workflows * chore: update workflows --------- Co-authored-by: kushal <kushalmahawar71@gmail.com> * chore: updated ci jobs (#18) * chore: added labeler * fix: fix minor issues * feat: #1 Auth Working and complete signup and login Funtionallity * fix : Fix Bugs or error * Fix : Env Variable Hided * Fix * init * chore: update workflows * chore: update workflows * chore: fix ci jobs * chore: Trigger CI re-run --------- Co-authored-by: kushal <kushalmahawar71@gmail.com> * Kushal (#19) * feat: #1 Auth Working and complete signup and login Funtionallity * fix : Fix Bugs or error * Fix : Env Variable Hided * Fix * init * chore: fix file name * chore: Trigger CI re-run * fix: updated package json * fix: added eslint * chore(web): run eslint on src and add minimal config * fix : eslint --------- Co-authored-by: Mohit Kumhar <mohitmolela@gmail.com> * chore: test (#10) * chore: fix authMiddlewere case-sensitive error * fix: conflicts * feat: Profile Section With All user detail Added * feat: Location Page Updated * fix: fix conflicts 2 * Fix conflicts --------- Co-authored-by: Mohit Kumhar <mohitmolela@gmail.com>

mohitkumhar and others added 6 commits September 23, 2025 16:49

chore: test (#9)

ff8b82c

chore: updated wordflows (#11)

4f21c32

* chore: added labeler * fix: fix minor issues * chore: testing jobs * chore: updated workflows

chore: updated ci jobs (#15)

4bbe88e

* chore: added labeler * fix: fix minor issues * chore: updated ci jobs

Copilot AI review requested due to automatic review settings September 24, 2025 04:27

kushalmahawar2005 requested review from mohitkumhar and removed request for Copilot September 24, 2025 04:27

kushalmahawar2005 assigned mohitkumhar Sep 24, 2025

kushalmahawar2005 requested review from Muskankhan512 and removed request for mohitkumhar September 24, 2025 05:49

Muskankhan512 previously approved these changes Sep 24, 2025

View reviewed changes

kushalmahawar2005 requested review from Copilot and mohitkumhar September 24, 2025 05:53

Copilot AI reviewed Sep 24, 2025

View reviewed changes

mohitkumhar changed the title ~~Main to Prod Merge~~ prod: Main to Prod Merge Sep 24, 2025

Fix: removed the duplicate folder (#22)

292ed19

* chore: test * chore: fixes

mohitkumhar dismissed Muskankhan512’s stale review via 292ed19 September 25, 2025 08:14

vercel Bot deployed to Production – shield-x-web-new September 25, 2025 08:17 View deployment

Copilot AI review requested due to automatic review settings September 25, 2025 17:24

Copilot AI reviewed Sep 25, 2025

vercel Bot deployed to Production – shield-x-web-new September 25, 2025 17:24 View deployment

chore: test (#24)

0dbf332

vercel Bot deployed to Production – shield-x-web-new September 25, 2025 17:34 View deployment

fix: typo (#25)

829fc17

Copilot AI review requested due to automatic review settings September 25, 2025 17:48

Copilot AI reviewed Sep 25, 2025

vercel Bot deployed to Production – shield-x-web-new September 25, 2025 17:48 View deployment

chore: added frontend content (#26)

c349d0e

vercel Bot deployed to Production – shield-x-web-new September 25, 2025 18:00 View deployment

vercel Bot deployed to Production – shield-x-web-slr4 September 25, 2025 18:07 View deployment

chore: added index.js and axios (#27)

2b6cf23

Copilot AI review requested due to automatic review settings September 25, 2025 18:20

Copilot AI reviewed Sep 25, 2025

vercel Bot deployed to Production – shield-x-web-new September 25, 2025 18:20 View deployment

vercel Bot deployed to Production – shield-x-web-slr4 September 25, 2025 18:20 View deployment

vercel Bot deployed to Production – shield-x-web-new September 25, 2025 18:48 View deployment

vercel Bot deployed to Production – shield-x-web-slr4 September 25, 2025 18:48 View deployment

mohitkumhar closed this Oct 6, 2025

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

prod: Main to Prod Merge #20

prod: Main to Prod Merge #20
kushalmahawar2005 wants to merge 13 commits into
prodfrom
main

kushalmahawar2005 commented Sep 24, 2025

Uh oh!

vercel Bot commented Sep 24, 2025 •

edited

Loading

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI Sep 24, 2025

Uh oh!

Copilot AI Sep 24, 2025

Uh oh!

Copilot AI Sep 24, 2025

Uh oh!

Copilot AI Sep 24, 2025

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI left a comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants



		<div style={{ textAlign: 'center', marginTop: 12 }}>
		<button className="link" type="button" onClick={() => navigate('/Signup')}>Don't have an account? <span style={{ color: 'red' }}>Sign Up</span></button>

		- POST /api/signup { name, email, password }
		- POST /api/login { email, password }

Conversation

kushalmahawar2005 commented Sep 24, 2025

Uh oh!

vercel Bot commented Sep 24, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Reviewed Changes

Uh oh!

Copilot AI Sep 24, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Sep 24, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Sep 24, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Sep 24, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

vercel Bot commented Sep 24, 2025 •

edited

Loading