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

Add confirmation for deleting a connection #304

Closed
5 tasks done
nevendyulgerov opened this issue Jan 16, 2025 · 0 comments · Fixed by #305
Closed
5 tasks done

Add confirmation for deleting a connection #304

nevendyulgerov opened this issue Jan 16, 2025 · 0 comments · Fixed by #305
Assignees
Labels
apps: web Type: Feature Add a new feature to the system

Comments

@nevendyulgerov
Copy link
Contributor

nevendyulgerov commented Jan 16, 2025

📄 Context

Currently, all you have to do to delete a connection is click it's delete button. Let's add a confirmation modal for this action to prevent cases where users click on the delete button by mistake.

📈 Subtasks

  • Add a confirmation that asks the user to confirm the delete action for a connection.
  • Cover the new logic with unit (and e2e) tests

🎯 Definition of Done

  • Deletion of connections need to be confirmed by users
  • All unit tests pass
  • All e2e tests pass
@nevendyulgerov nevendyulgerov added Type: Feature Add a new feature to the system apps: web labels Jan 16, 2025
@github-project-automation github-project-automation bot moved this to 📌 Todo in Explorer Unit Jan 16, 2025
@nevendyulgerov nevendyulgerov self-assigned this Jan 16, 2025
@nevendyulgerov nevendyulgerov moved this from 📌 Todo to 🧑‍💻 In Progress in Explorer Unit Jan 16, 2025
@nevendyulgerov nevendyulgerov linked a pull request Jan 16, 2025 that will close this issue
@nevendyulgerov nevendyulgerov moved this from 🧑‍💻 In Progress to 👁 In Review in Explorer Unit Jan 17, 2025
@github-project-automation github-project-automation bot moved this from 👁 In Review to 📦 Done in Explorer Unit Jan 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
apps: web Type: Feature Add a new feature to the system
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

1 participant