feat(monitors,workergroups): implement deletion and stauts changes

internal/handlers/settingsmonitors.go

@@ -123,6 +123,54 @@ func (h *BaseHandler) SettingsMonitorsDescribeGET(c echo.Context) error {
 	})
 }
 
+func (h *BaseHandler) SettingsMonitorsDescribeDELETE(c echo.Context) error {
+	slug := c.Param("slug")
+
+	err := services.DeleteMonitor(context.Background(), h.db, slug)
+	if err != nil {
+		return err
+	}
+
+	err = services.DeleteMonitorSchedule(context.Background(), h.temporal, slug)
+	if err != nil {
+		return err
+	}
+
+	return c.Redirect(http.StatusSeeOther, "/settings/monitors")
+}
+
+func (h *BaseHandler) SettingsMonitorsDisableGET(c echo.Context) error {
+	slug := c.Param("slug")
+
+	monitor, err := services.GetMonitor(context.Background(), h.db, slug)
+	if err != nil {
+		return err
+	}
+
+	err = services.SetMonitorStatus(context.Background(), h.temporal, monitor.Slug, services.MonitorStatusPaused)
+	if err != nil {
+		return err
+	}
+
+	return c.Redirect(http.StatusSeeOther, fmt.Sprintf("/settings/monitors/%s", slug))
+}
+
+func (h *BaseHandler) SettingsMonitorsEnableGET(c echo.Context) error {
+	slug := c.Param("slug")
+
+	monitor, err := services.GetMonitor(context.Background(), h.db, slug)
+	if err != nil {
+		return err
+	}
+
+	err = services.SetMonitorStatus(context.Background(), h.temporal, monitor.Slug, services.MonitorStatusActive)
+	if err != nil {
+		return err
+	}
+
+	return c.Redirect(http.StatusSeeOther, fmt.Sprintf("/settings/monitors/%s", slug))
+}
+
 func (h *BaseHandler) SettingsMonitorsDescribePOST(c echo.Context) error {
 	ctx := context.Background()
 	monitorSlug := c.Param("slug")

internal/handlers/settingsworkergroups.go

@@ -108,6 +108,17 @@ func (h *BaseHandler) SettingsWorkerGroupsDescribeGET(c echo.Context) error {
 	})
 }
 
+func (h *BaseHandler) SettingsWorkerGroupsDescribeDELETE(c echo.Context) error {
+	slug := c.Param("slug")
+
+	err := services.DeleteWorkerGroup(context.Background(), h.db, slug)
+	if err != nil {
+		return err
+	}
+
+	return c.Redirect(http.StatusSeeOther, "/settings/worker-groups")
+}
+
 func (h *BaseHandler) SettingsWorkerGroupsCreateGET(c echo.Context) error {
 	cc := c.(AuthenticatedContext)
 

internal/services/monitor.go

@@ -40,6 +40,20 @@ func GetMonitorStatus(ctx context.Context, temporal client.Client, slug string)
 	return MonitorStatusActive, nil
 }
 
+func SetMonitorStatus(ctx context.Context, temporal client.Client, slug string, status MonitorStatus) error {
+	schedule := temporal.ScheduleClient().GetHandle(ctx, getScheduleId(slug))
+
+	if status == MonitorStatusActive {
+		return schedule.Unpause(ctx, client.ScheduleUnpauseOptions{Note: "Unpaused by user"})
+	}
+
+	if status == MonitorStatusPaused {
+		return schedule.Pause(ctx, client.SchedulePauseOptions{Note: "Paused by user"})
+	}
+
+	return nil
+}
+
 func CreateMonitor(ctx context.Context, db *sqlx.DB, monitor *models.Monitor) error {
 	_, err := db.NamedExecContext(ctx,
 		"INSERT INTO monitors (slug, name, script, schedule) VALUES (:slug, :name, :script, :schedule)",
@@ -56,6 +70,14 @@ func UpdateMonitor(ctx context.Context, db *sqlx.DB, monitor *models.Monitor) er
 	return err
 }
 
+func DeleteMonitor(ctx context.Context, db *sqlx.DB, slug string) error {
+	_, err := db.ExecContext(ctx,
+		"UPDATE monitors SET deleted_at = datetime('now') WHERE slug=$1",
+		slug,
+	)
+	return err
+}
+
 func UpdateMonitorWorkerGroups(ctx context.Context, db *sqlx.DB, monitor *models.Monitor, workerGroups []*models.WorkerGroup) error {
 	tx, err := db.BeginTxx(ctx, nil)
 	if err != nil {
@@ -204,6 +226,11 @@ ORDER BY monitors.name
 	return maps.Values(monitors), err
 }
 
+func DeleteMonitorSchedule(ctx context.Context, t client.Client, slug string) error {
+	schedule := t.ScheduleClient().GetHandle(ctx, getScheduleId(slug))
+	return schedule.Delete(ctx)
+}
+
 func CreateOrUpdateMonitorSchedule(
 	ctx context.Context,
 	t client.Client,

internal/services/worker_group.go

@@ -32,6 +32,14 @@ func CreateWorkerGroup(ctx context.Context, db *sqlx.DB, workerGroup *models.Wor
 	return err
 }
 
+func DeleteWorkerGroup(ctx context.Context, db *sqlx.DB, slug string) error {
+	_, err := db.ExecContext(ctx,
+		"UPDATE worker_groups SET deleted_at = datetime('now') WHERE slug = $1",
+		slug,
+	)
+	return err
+}
+
 func GetWorkerGroups(ctx context.Context, db *sqlx.DB) ([]*models.WorkerGroup, error) {
 	var workerGroups []*models.WorkerGroup
 	err := db.SelectContext(ctx, &workerGroups,

pkg/server/routes.go

@@ -50,10 +50,15 @@ func Routes(
 	settings.POST("/monitors/create", h.SettingsMonitorsCreatePOST)
 	settings.GET("/monitors/:slug", h.SettingsMonitorsDescribeGET)
 	settings.POST("/monitors/:slug", h.SettingsMonitorsDescribePOST)
+	settings.GET("/monitors/:slug/delete", h.SettingsMonitorsDescribeDELETE)
+	settings.GET("/monitors/:slug/disable", h.SettingsMonitorsDisableGET)
+	settings.GET("/monitors/:slug/enable", h.SettingsMonitorsEnableGET)
 	settings.GET("/worker-groups", h.SettingsWorkerGroupsGET)
 	settings.GET("/worker-groups/create", h.SettingsWorkerGroupsCreateGET)
 	settings.POST("/worker-groups/create", h.SettingsWorkerGroupsCreatePOST)
 	settings.GET("/worker-groups/:slug", h.SettingsWorkerGroupsDescribeGET)
+	settings.GET("/worker-groups/:slug/delete", h.SettingsWorkerGroupsDescribeDELETE)
+
 	settings.Match([]string{"GET", "HEAD", "POST", "PUT", "PATCH", "DELETE"}, "/temporal*", h.Temporal)
 
 	// OAuth2

pkg/server/server.go

@@ -46,7 +46,7 @@ func (s *Server) Start() error {
 	s.worker = NewWorker(temporalClient, s.cfg)
 
 	s.echo.Renderer = templates.NewTemplates()
-	//s.echo.Use(middleware.Logger())
+	s.echo.Use(middleware.Logger())
 	s.echo.Use(middleware.Recover())
 	Routes(s.echo, db, temporalClient, s.cfg, s.logger)
 

web/templates/pages/settings_monitors_describe.tmpl

@@ -40,9 +40,16 @@
     </span>
     {{ end }}
     </h2>
-    <p class="text-sm mb-2">Pausing the monitor will stop it from executing. This can be useful in cases of expected downtime.</p>
-    <a class="block text-center py-2.5 px-5 me-2 mb-2 text-sm font-medium text-gray-900 focus:outline-none bg-white rounded-lg border border-gray-200 hover:bg-gray-100 hover:text-blue-700 focus:z-10 focus:ring-4 focus:ring-gray-100" href="/settings/monitors/{{ .Monitor.Slug }}/enable">Pause</a>
-    <a class="block text-center py-2.5 px-5 me-2 mb-2 text-sm font-medium text-gray-900 focus:outline-none bg-white rounded-lg border border-gray-200 hover:bg-gray-100 hover:text-blue-700 focus:z-10 focus:ring-4 focus:ring-gray-100" href="/settings/monitors/{{ .Monitor.Slug }}/disable">Resume</a>
+    <p class="text-sm mb-2">
+      Pausing the monitor will stop it from executing.
+      This can be useful in cases of expected downtime.
+      Or when the monitor is not needed anymore.
+    </p>
+    {{ if eq .Monitor.Status "ACTIVE" }}
+      <a class="block text-center py-2.5 px-5 me-2 mb-2 text-sm font-medium text-gray-900 focus:outline-none bg-white rounded-lg border border-gray-200 hover:bg-gray-100 hover:text-blue-700 focus:z-10 focus:ring-4 focus:ring-gray-100" href="/settings/monitors/{{ .Monitor.Slug }}/disable">Pause</a>
+    {{ else if eq .Monitor.Status "PAUSED" }}
+      <a class="block text-center py-2.5 px-5 me-2 mb-2 text-sm font-medium text-gray-900 focus:outline-none bg-white rounded-lg border border-gray-200 hover:bg-gray-100 hover:text-blue-700 focus:z-10 focus:ring-4 focus:ring-gray-100" href="/settings/monitors/{{ .Monitor.Slug }}/enable">Resume</a>
+    {{ end }}
   </section>
 
   <section class="p-2 flex-1 border-4 border-red-300">