Skip to content

Commit

Permalink
Merge pull request #61 from uc-cdis/chore/vadc-1318
Browse files Browse the repository at this point in the history
Backout temp fix
  • Loading branch information
tianj7 authored Oct 10, 2024
2 parents 4107a4b + 98b2a9d commit e38e833
Showing 1 changed file with 0 additions and 5 deletions.
5 changes: 0 additions & 5 deletions js/pages/cohort-definitions/cohort-definition-manager.js
Original file line number Diff line number Diff line change
Expand Up @@ -410,11 +410,6 @@ define(['jquery', 'knockout', 'text!./cohort-definition-manager.html',
(this.isAuthenticated() && this.authApi.isPermittedCopyCohort(this.currentCohortDefinition().id()) || !config.userAuthenticationEnabled);
});
this.canDelete = ko.pureComputed(() => {
// tmp: temporarily disable the delete option for cohortdefinition
// until we have a final solution for storing cohortdefinitions as part
// of a workflow run. TODO - remove this once a solution is in place.
// See https://ctds-planx.atlassian.net/browse/VADC-1177
return false;
if (this.isNew()) {
return false;
}
Expand Down

0 comments on commit e38e833

Please sign in to comment.