[JN-565] fix trigger editing for inactive triggers (#1287)

api-admin/src/main/java/bio/terra/pearl/api/admin/service/notifications/TriggerExtService.java

@@ -125,6 +125,9 @@ public Trigger replace(PortalStudyEnvAuthContext authContext, UUID configId, Tri
 
     Trigger existing = triggerService.find(configId).get();
     verifyTrigger(authContext, existing);
+    if (!existing.isActive()) {
+      throw new IllegalArgumentException("Cannot edit inactive trigger");
+    }
 
     Trigger newConfig = create(update, authContext.getStudyEnvironment(), portalEnvironment);
     // after creating the new config, deactivate the old config
@@ -186,6 +189,10 @@ private void verifyTrigger(PortalStudyEnvAuthContext authContext, Trigger config
 
   private Trigger create(
       Trigger newConfig, StudyEnvironment studyEnvironment, PortalEnvironment portalEnvironment) {
+    if (!newConfig.isActive()) {
+      throw new IllegalArgumentException("Cannot create inactive trigger");
+    }
+
     newConfig.cleanForCopying();
     newConfig.setStudyEnvironmentId(studyEnvironment.getId());
     newConfig.setPortalEnvironmentId(portalEnvironment.getId());

ui-admin/src/study/notifications/TriggerDesigner.tsx

@@ -84,6 +84,10 @@ export const TriggerDesigner = (
   }
 
   return <div>
+    {!workingTrigger.active &&
+        <div className="alert alert-warning mt-0">
+            This trigger is no longer active. Inactive triggers cannot be edited.
+        </div>}
 
     <TriggerDesignerEditor
       studyEnvContext={studyEnvContext}
@@ -93,8 +97,17 @@ export const TriggerDesigner = (
     />
 
     <div className="d-flex justify-content-center mt-2">
-      <button type="button" className="btn btn-primary" onClick={saveConfig}>Save</button>
-      <button type="button" className="btn btn-danger ms-4" onClick={() => setShowDeleteModal(true)}>Delete</button>
+      <button type="button" className="btn btn-primary" disabled={!workingTrigger.active} onClick={saveConfig}>
+        Save
+      </button>
+      <button
+        type="button"
+        className="btn btn-danger ms-4"
+        disabled={!workingTrigger.active}
+        onClick={() => setShowDeleteModal(true)}
+      >
+        Delete
+      </button>
     </div>
 
     {showDeleteModal && (

ui-admin/src/study/notifications/TriggerList.tsx

@@ -104,7 +104,7 @@ export default function TriggerList({ studyEnvContext, portalContext }:
           </li> }
         </ul>
       </div> }
-      <div className="flex-grow-1 bg-white p-3">
+      <div className="flex-grow-1 bg-white px-3">
         <Routes>
           <Route path="triggers/:triggerId"
             element={<TriggerDesigner studyEnvContext={studyEnvContext}