Skip to content

Neeraj Fix Job Application Form#5228

Open
Neeraj-Kondaveeti wants to merge 1 commit into
developmentfrom
Neeraj_Fix_Job_Application_Form
Open

Neeraj Fix Job Application Form#5228
Neeraj-Kondaveeti wants to merge 1 commit into
developmentfrom
Neeraj_Fix_Job_Application_Form

Conversation

@Neeraj-Kondaveeti
Copy link
Copy Markdown
Contributor

@Neeraj-Kondaveeti Neeraj-Kondaveeti commented May 5, 2026

Description

image image image

Related PRS (if any):

This frontend PR is related to the development backend branch.

Main changes explained:

  • Updated JobApplicationForm.jsx

    • Implemented normalized deduplication using normalizeTitleKey to remove duplicate questions
    • Added real-time and submit validation for numeric-only input for "Hours per week"
    • Replaced text input with dropdown for "Individual vs Organization" question
    • Ensured only dropdown renders (removed textarea/textbox override for this question)
    • Integrated native date picker (type="date") for "Start Date"
    • Added inline validation error messages and visual feedback (red border)
  • Updated JobApplicationForm.module.css

    • Adjusted styling for improved layout and responsiveness
    • Fixed dropdown appearance issues (including arrow duplication)

How to test:

  1. check into current branch
  2. do npm install and ... to run this PR locally
  3. Clear site data/cache
  4. log as admin user
  5. Navigate to: http://localhost:5173/job-application
  6. Verify the following:
    • Logo size is properly reduced
    • Top-right dropdown displays correctly (no duplicate arrows)
    • First Name and Last Name fields are separate
    • "Are you applying as an individual or organization?" shows a dropdown
    • "Hours per week" field:
      • Shows error for non-numeric input
    • "Start Date" shows a calendar picker
    • No duplicate questions are displayed

Screenshots or videos of changes:

image image image image

Note:

N/A

@netlify
Copy link
Copy Markdown

netlify Bot commented May 5, 2026

Deploy Preview for highestgoodnetwork-dev ready!

Name Link
🔨 Latest commit 458f222
🔍 Latest deploy log https://app.netlify.com/projects/highestgoodnetwork-dev/deploys/69f956fc9a215e00086bc80e
😎 Deploy Preview https://deploy-preview-5228--highestgoodnetwork-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

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

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented May 5, 2026

@Neeraj-Kondaveeti Neeraj-Kondaveeti added the High Priority - Please Review First This is an important PR we'd like to get merged as soon as possible label May 5, 2026
Copy link
Copy Markdown

@kunchalasireesha kunchalasireesha left a comment

Choose a reason for hiding this comment

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

PR-5228.mov

Hi Neeraj, the local testing looks great! One quick thing: Sonar is flagging an error in JobApplicationForm.jsx. Can you clear that up before the merge? Thanks!"

Copy link
Copy Markdown
Contributor

@sundarmachani sundarmachani left a comment

Choose a reason for hiding this comment

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

Tested the changes locally by checking into the current branch, running npm install, and starting the application. Cleared site data/cache, logged in as an admin user, and navigated to http://localhost:5173/job-application. Verified that the logo size is properly reduced, the top-right dropdown displays correctly without duplicate arrows, the First Name and Last Name fields are separated, and the “Are you applying as an individual or organization?” field is shown as a dropdown. Also confirmed that the “Hours per week” field displays validation errors for non-numeric input, the “Start Date” field shows a calendar picker correctly, and no duplicate questions are displayed. All requested changes are working as expected locally. Attached screenshots for verification.

Image Image Image

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

Labels

High Priority - Please Review First This is an important PR we'd like to get merged as soon as possible

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants