search

stats4sd / aec_portfolio

A proof of concept for the AEC Consortium Project Management / Assessment System
GNU General Public License v3.0
0 stars 0 forks source link

Add soft delete #139

Closed dan-tang-ssd closed 1 year ago

dan-tang-ssd commented 1 year ago

This PR fixes #113 .

This is submitted for adding soft-delete to portfolios and projects. It also add ON DELETE CASCADE ON UPDATE CASCADE for projects.portfolio_id.

There are two remaining items:

  1. After confirming to remove a portfolio, it shows a white page with a "1". Suppose it should divert to portfolios page.
  2. I do not know how to add a working "Remove Initiative" button in Vue component.
dan-tang-ssd commented 1 year ago

I am trying to add a "Remove Initiative" button, both buttons are not working yet...

image

image

dave-mills commented 1 year ago

@dan-tang-ssd I've updated the vue component. I used a trick we've used in the past, where the 'prop' becomes the "initial" read-only version of a property, and onMount() it's set to a reactive variable. Looks like it's working ok now.

dan-tang-ssd commented 1 year ago

It works well. That's great. Thank you Dave.

dan-tang-ssd commented 1 year ago

Live env deployment is completed. A quick hot fix is deployed shortly to resolve duplicate import statement issue.

Performed testing in with positive result in live env. An initiative can be removed now.