Skip to content

Conversation

@20luky3
Copy link
Contributor

@20luky3 20luky3 commented Jan 6, 2026

TL;DR

Adds the destroy route in ProjectsController


What is this PR trying to achieve?

-closes #71


How did you achieve it?

  • Add destroy route to projects_controller
  • Add a delete project button to the project's show page which shows a confirmation popup
  • Update controller tests to include destroy and add a specific destroy project test
image

Checklist

  • Changes have been top-hatted locally
  • Tests have been added or updated
  • Documentation has been updated (if applicable)
  • Linked related issues

@20luky3 20luky3 requested a review from CulmoneY January 8, 2026 01:14
@CulmoneY CulmoneY force-pushed the yan/71-delete-project branch from 2545da8 to ce8955f Compare January 9, 2026 01:36
@CulmoneY CulmoneY merged commit 1e84415 into main Jan 9, 2026
6 checks passed
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.

ProjectsController: Delete project

3 participants