Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added delete confirmation for managing projects #203

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

aubreyfloyd2
Copy link

Issue: When a user is on the "Manage Projects" pop-up from their project dashboard, clicking the "Delete" button for any project immediately deletes the project without any confirmation prompt. This increases the risk of accidental data loss, as users might click "Delete" unintentionally. (#200)

Pull Request: Here a confirmation step has been added to prevent accidental deletions and for better user experience.

Changes Implemented:

  • Added a confirmation pop-up when the "Delete" button is clicked.
  • Added showConfirmation to manage when the confirmation prompt is shown.
  • Created confirmDelete, deleteProject, and cancelDelete functions to handle what happens each for the user.
  • Updated the template to include the confirmation pop-up, with "Delete" and "Cancel" buttons.
  • Modified the delete logic to only proceed with deletion after user confirmation.

Testing:

  • Successfully tested the new confirmation for the "Delete" button.
  • Confirmation pop-up appears before project deletion.
  • Deletion only happens after the user confirms the action.
  • Clicking "Cancel" stops the deletion and keeps the project on the board.

Screenshots show an example of one test: before deletion, confirmation popup, and after deletion:
Screenshot 2024-12-02 at 11 04 33 AM
Screenshot 2024-12-02 at 11 13 23 AM
Screenshot 2024-12-02 at 11 13 30 AM
Screenshot 2024-12-02 at 11 13 41 AM

Copy link

vercel bot commented Dec 2, 2024

@aubreyfloyd2 is attempting to deploy a commit to the merkie's projects Team on Vercel.

A member of the Team first needs to authorize it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant