Purav taking over for Sohail - Fix job form issues frontend - #5469
Open
bfire9989 wants to merge 2 commits into
Open
Purav taking over for Sohail - Fix job form issues frontend#5469bfire9989 wants to merge 2 commits into
bfire9989 wants to merge 2 commits into
Conversation
✅ Deploy Preview for highestgoodnetwork-dev ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.



Description
Fixes # (Priority High) — Job Application Page (
/job-application) form functionality issues reported by Sayantan (WIP Sohail Uddin Syed). Related previous PR: #4168.Several bugs were found on the main branch when testing the job application page. This PR addresses all frontend-side issues identified.
Related PRs
This frontend PR is related to the #2318 backend PR.
Main changes explained
JobApplicationForm.jsx— required fields (Name, Email, and form questions marked required) now display*in their labels/placeholdersJobApplicationForm.jsx— all form fields (name, email, location, phone, company, website, resume, answers) are reset after a successful submissionJobApplicationForm.jsx— "Click to know more about this position" now navigates to the job's externaljobDetailsLinkwhen available; falls back to an inline description modal when no link existsJobApplicationForm.jsx— submission payload switched from JSON tomultipart/form-datato support optional resume file uploadJobApplicationForm.jsx—409response from backend is caught and shown as "Application already submitted" toast, preventing duplicate submissionsJobApplicationForm.jsx— position dropdown commented out temporarily as it is not retrieving form data as expected; search by job title input remains functionalJobApplicationForm.module.css— addedlinkButtonstyle for the accessible fallback "know more" triggerjsx-a11y/anchor-is-validlint error — replaced<a href="#">used as a button with a proper<button>elementHow to test
npm installthen start the dev server/job-application*on Name, Email, and required questionsjobDetailsLinkexists, or opens the description modal when it does notmultipart/form-datawith aresumefield)Screenshots or videos of changes
Note
The position dropdown in the top-right is intentionally commented out pending a fix to how it retrieves and maps form data. The job title search input on the left remains fully functional as an alternative.