Resume Craft: Your go-to platform for effortlessly tailoring ATS-friendly resumes to job descriptions. Streamline your job application process and stand out in the competitive market.
Move the confirmation dialog from the page component to the section components (because we need to show different dialogs on different button clicks. State management was becoming difficult and was unnecessarily increasing the code in the page component. So, need to move it into the individual sections where it is being used.)
Changes made:
Moved the Confirmation dialog from the page to a separate component DeleteConfirmationDialog.
Added tests for that component
Used this component inside the modals and managing its states inside these individual components.
How Has This Been Tested?
Tested the UI manually
Wrote tests for some components
Screenshots / Videos (If required)
Checklist:
[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
Summary
Requirement:
Move the confirmation dialog from the page component to the section components (because we need to show different dialogs on different button clicks. State management was becoming difficult and was unnecessarily increasing the code in the page component. So, need to move it into the individual sections where it is being used.)
Changes made:
DeleteConfirmationDialog
.How Has This Been Tested?
Screenshots / Videos (If required)
Checklist: