Repository navigation
feat: Added Dynamic Resume Completion Progress - #69
prachi-phatak wants to merge 1 commit into
Conversation
|
@prachi-phatak is attempting to deploy a commit to the Pari's projects Team on Vercel. A member of the Team first needs to authorize it. |
|
Hi , I am a contributor in ECSoC26. Will this contribution be eligible for ECSoC26 points? Thank you! |
pari-28
left a comment
There was a problem hiding this comment.
Hi @prachi-phatak,
Thank you for your contribution! The Resume Completion Progress feature is a useful addition and aligns well with the goals of Resume Studio.
Before this PR can be merged, I'd like to request a few changes:
The current branch does not pass the lint and build checks. Please resolve these issues and ensure all CI checks pass.
There appear to be duplicated sections of code in ResumeStudio.tsx (such as repeated checks declarations and duplicated logic). Please clean up any merge artifacts or redundant code before resubmitting.
This PR also modifies package.json and package-lock.json. Since this feature does not appear to require additional dependencies, please remove any unnecessary package changes or explain why they are required.
If possible, try to reuse existing validation logic where appropriate instead of duplicating similar checks.
Once these points are addressed, I'll be happy to review the updated implementation again.
Pull Request
📌 Feature: Added Dynamic Resume Completion Progress
Closes #34
📝 Description
This PR implements a dynamic Resume Completion Progress feature in Resume Studio. The progress bar updates in real time based on completed resume sections and highlights missing sections to help users build a complete resume.
✨ Changes Made
🚀 Type of Change
🧪 Testing
📸 Screenshots
✅ Checklist