Skip to content

[FEATURE] Application Confirmation Modal Component#7

Merged
suvajitcodewalnut merged 4 commits intomasterfrom
feature/application-confirmation-modal
Nov 7, 2025
Merged

[FEATURE] Application Confirmation Modal Component#7
suvajitcodewalnut merged 4 commits intomasterfrom
feature/application-confirmation-modal

Conversation

@suvajitcodewalnut
Copy link
Copy Markdown
Owner

@suvajitcodewalnut suvajitcodewalnut commented Nov 7, 2025

QUICKLAB : PULL REQUEST

DESCRIPTION

  • Type: ( Feature )
  • Related Issue(s): N/A
  • This implements Confirmation Modal with proper state lifting and unit tests.

IMPLEMENTATION DETAILS

  • This implements Confirmtaion Modal with proper Confirmation and Close button.
  • Written unit-testing for Confirmation Modal.
  • Implemented proper state lifting.

TESTING WALKTHROUGH

  • Run the application with bun run application:dev command.
  • Verify that for each card component, we have the Delete button.
  • Click on the Delete button and verify that Confirmation Modal pops us.

CHECKLIST

  • Codebase is properly formatted and adheres to the project’s style guide.
  • Codebase passes all linting checks without warnings or errors.
  • Relevant unit tests have been written and all tests pass successfully.
  • Branch is up to date with the latest changes from the base branch.

@suvajitcodewalnut suvajitcodewalnut merged commit 9b0e1cb into master Nov 7, 2025
4 checks passed
@suvajitcodewalnut suvajitcodewalnut deleted the feature/application-confirmation-modal branch November 7, 2025 19:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant