From b60668b9e8386ec2a00e744b952bee2c394149a1 Mon Sep 17 00:00:00 2001
From: aegbert <aegbert@lucidchart.com>
Date: Fri, 7 Nov 2025 10:09:41 -0700
Subject: [PATCH] Add warning messages to UI changes

---
 .../piezo/admin/views/job.scala.html          | 22 ++++++++++++++-----
 .../piezo/admin/views/trigger.scala.html      |  7 ++++++
 admin/conf/application.conf                   |  2 ++
 3 files changed, 25 insertions(+), 6 deletions(-)

diff --git a/admin/app/com/lucidchart/piezo/admin/views/job.scala.html b/admin/app/com/lucidchart/piezo/admin/views/job.scala.html
index dccced7..cc285cc 100644
--- a/admin/app/com/lucidchart/piezo/admin/views/job.scala.html
+++ b/admin/app/com/lucidchart/piezo/admin/views/job.scala.html
@@ -66,13 +66,23 @@ <h4>Are you sure you want to run this job right now?</h4>
             <button type="button" class="btn btn-default" onclick="$('#runConfirm').hide()">Cancel</button>
         </div>
 
-        <!-- Delete Job Alerts -->
-        <div id="deleteConfirm" style="display:none;" class="alert alert-warning fade in">
-          <h4>This will permanently delete the job!</h4>
-          <button type="button" class="btn btn-danger" onclick="deleteJob()">Delete</button>
-          <button type="button" class="btn btn-default" onclick="$('#deleteConfirm').hide()">Cancel</button>
-        </div>
         @defining(configuration.getOptional[Boolean]("com.lucidchart.piezo.admin.production").contains(true)) { inProd =>
+            @defining(configuration.getOptional[String]("com.lucidchart.piezo.admin.confirmDeleteMessage").getOrElse("This will permanently delete the job!")) { confirmDeleteMessage =>
+                <!-- Delete Job Alerts -->
+                <div id="deleteConfirm" style="display:none;" class="alert alert-@if(inProd) {danger} else {warning} fade in">
+                    <h4>@(confirmDeleteMessage)</h4>
+                    <button type="button" class="btn btn-danger" onclick="deleteJob()">Delete</button>
+                    <button type="button" class="btn btn-default" onclick="$('#deleteConfirm').hide()">Cancel</button>
+                </div>
+            }
+            @defining(configuration.getOptional[String]("com.lucidchart.piezo.admin.confirmEditMessage").getOrElse("This will modify the job!")) { confirmEditMessage =>
+                <!-- Edit Job Alerts -->
+                <div id="editConfirm" style="display:none;" class="alert alert-@if(inProd) {danger} else {warning} fade in">
+                    <h4>@(confirmEditMessage)</h4>
+                    <button type="button" class="btn btn-danger" onclick="editTrigger()">Edit</button>
+                    <button type="button" class="btn btn-default" onclick="$('#editConfirm').hide()">Cancel</button>
+                </div>
+            }
             <!-- Pause Job Alerts -->
             <div id="pauseConfirm" style="display: none;" class="alert alert-@if(inProd) {danger} else {warning} fade in">
                 <h4>Are you sure you want to pause the job?</h4>
diff --git a/admin/app/com/lucidchart/piezo/admin/views/trigger.scala.html b/admin/app/com/lucidchart/piezo/admin/views/trigger.scala.html
index 31fb19a..d225db1 100644
--- a/admin/app/com/lucidchart/piezo/admin/views/trigger.scala.html
+++ b/admin/app/com/lucidchart/piezo/admin/views/trigger.scala.html
@@ -54,11 +54,18 @@ <h3 class="pull-left">@currentTrigger.get.getKey.getGroup() &raquo; <span class=
             <p id="errorMessage"></p>
         </div>
 
+        <!-- Delete Trigger Alert -->
         <div id="deleteConfirm" style="display:none;" class="alert alert-warning fade in">
           <h4>This will permanently delete the trigger!</h4>
             <button type="button" class="btn btn-danger" onclick="deleteTrigger()">Delete</button>
             <button type="button" class="btn btn-default" onclick="$('#deleteConfirm').hide()">Cancel</button>
         </div>
+        <!-- Modify Trigger Alert -->
+        <div id="editConfirm" style="display:none;" class="alert alert-warning fade in">
+            <h4>This will permanently delete the trigger!</h4>
+            <button type="button" class="btn btn-danger" onclick="editTrigger()">Edit</button>
+            <button type="button" class="btn btn-default" onclick="$('#editConfirm').hide()">Cancel</button>
+        </div>
         <script>
 
             function showErrorMessage(message) {
diff --git a/admin/conf/application.conf b/admin/conf/application.conf
index 53b8f51..9949d04 100644
--- a/admin/conf/application.conf
+++ b/admin/conf/application.conf
@@ -44,6 +44,8 @@ i18n.langs = ["en"]
 # evolutionplugin=disabled
 com.lucidchart.piezo.heartbeatFile="/tmp/piezo/workerHeartbeatFile"
 com.lucidchart.piezo.admin.production=false
+com.lucidchart.piezo.admin.confirmDeleteMessage="This will permanently delete the job!"
+com.lucidchart.piezo.admin.confirmEditMessage="This will modify the job!"
 healthCheck.worker.minutesBetween=5
 play.application.loader=com.lucidchart.piezo.admin.PiezoAdminApplicationLoader