search

fpdcc / ccfp-asset-dashboard

CCFP Asset Dashboard
0 stars 1 forks source link

project delete #218

Closed glw closed 1 year ago

glw commented 2 years ago

We forgot to include wholesale removal of projects. Not exactly sure where in the dashboard this should be implemented. That may take a conversation to figure out.

At the moment we can remove projects via the admin page.

smcalilly commented 1 year ago

@xmedr For a V1, we should include a delete button on the project detail page. Don't sweat over the location of the button right now — we can figure out where it feels best after using it for a bit.

The delete mechanism should be similar to the flow for deleing a phase:

  1. user clicks delete
  2. redirects to a confirmation page
  3. user confirms the deletion and submits the form to a DeleteView (you can see how we've done the PhaseDeleteView for inspiration)
  4. the success url redirect should take the user to the root page of the app (the page with the table of projects)
smcalilly commented 1 year ago

@xmedr has implemented all of the views / templates, but this is block until @smcalilly cleans up the django signals in #224

smcalilly commented 1 year ago

@xmedr this is unblocked now that I've merged PR #235. There are probably some big changes in there that might affect your code. We'll need to rebase your branch with master. Let me know if you need help with that.