Skip to content

Sriamsh_Enforce_Production_Identity_for_Dev_Account_Creation_Frontend - #4445

Closed
Sriamshreddy000 wants to merge 3 commits into
developmentfrom
sriamsh_prod_identity_frontend
Closed

Sriamsh_Enforce_Production_Identity_for_Dev_Account_Creation_Frontend#4445
Sriamshreddy000 wants to merge 3 commits into
developmentfrom
sriamsh_prod_identity_frontend

Conversation

@Sriamshreddy000

Copy link
Copy Markdown
Contributor

Description

Screenshot 2025-11-23 at 2 06 09 PM https://docs.google.com/document/d/1rZDo_yYs5rUyHDkA_3HfPxdiOLaRzmQ509gr6QA2kys/edit?tab=t.0

Related PRS (if any):

This frontend PR is related to the #1918 backend PR.
To test this backend PR you need to checkout the #1918 frontend PR.

Main changes explained:

  • Added new route for /dev-signup in routes.jsx
  • Created DevSignup.jsx page with the required input fields
  • Implemented form validation (required fields + password rules)
  • Integrated API request to /api/dev/signup-production
  • Displayed backend error messages and success messages
  • Added button disable + UI feedback states

How to test:

  1. check into current branch
  2. do npm install and ... to run this PR locally
  3. Checkout the related frontend branch and start the frontend.
  4. Clear site data/cache
  5. Attempt a new Dev signup by navigating to /dev-signup:
    • Enter valid mock Production credentials → expect the Dev account to be created.
    • Enter invalid Production credentials (for example, reusing the same Prod email twice) → expect an error message and a new log entry.
  6. Open MongoDB Compass and check:
    • if a new entry is created in userProfiles upon successful account creation
    • if a new entry log is created in devSignupLogs upon unsuccessful creation

Screenshots or videos of changes:

Please find screenshots in PR #1918

Note:

This PR only implements frontend logic based on mock Production validation.
Actual Production API integration can be plugged in later with no UI changes required.

@netlify

netlify Bot commented Nov 23, 2025

Copy link
Copy Markdown

Deploy Preview for highestgoodnetwork-dev ready!

Name Link
🔨 Latest commit 55109ff
🔍 Latest deploy log https://app.netlify.com/projects/highestgoodnetwork-dev/deploys/69239cb6953ad200087443c7
😎 Deploy Preview https://deploy-preview-4445--highestgoodnetwork-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@sonarqubecloud

Copy link
Copy Markdown

@VijayAnirudh VijayAnirudh left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

image Followed the exact steps mentioned. However, this branch requires 14.xx, but as per the document, I'm on V 20.xx.

@one-community one-community added the High Priority - Please Review First This is an important PR we'd like to get merged as soon as possible label Dec 16, 2025
@Sriamshreddy000

Copy link
Copy Markdown
Contributor Author

Followed the exact steps mentioned. However, this branch requires 14.xx, but as per the document, I'm on V 20.xx.

Hi VijayAnirudh,
Yes, as per the setup instructions, we’re expected to use Node 20.x.x, and I also built and tested this branch locally using Node 20.x.x. I ran npm install again on this PR branch and it completed successfully without any blocking errors.

The issue you’re seeing appears to be related to local lockfile state. You may want to try removing the existing lockfiles and running npm install again under Node 20 so the dependencies are regenerated cleanly.

Please let me know if you still encounter any issues.

@sayali-2308 sayali-2308 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cannot test frontend PR without backend.
Frontend PR #4445 requires backend PR #1918 to be available for testing. Backend branch sriamsh_prod_identity_backend does not exist in remote repository. Cannot proceed with functional testing until backend code is pushed and available.
Waiting for backend PR #1918 to be accessible before continuing frontend review.

@Anusha-Gali Anusha-Gali left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi Sriamsh,

I have reviewed your PR locally and was able to verify all requirements. However the page does not align with dark mode.
Image
Image
Image

@naznin07 naznin07 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add loading state to spam the submit button.

PR # 4445

@one-community one-community added Needs New Developer This is a PR that is partially developed but needs someone new to take it over and finish it. do not review Do not review or look at code without full context and removed High Priority - Please Review First This is an important PR we'd like to get merged as soon as possible labels Apr 12, 2026
@one-community

Copy link
Copy Markdown
Member

Replaced by this set: #5420 +2289

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

Labels

do not review Do not review or look at code without full context Needs New Developer This is a PR that is partially developed but needs someone new to take it over and finish it.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants