send test notification in notifier adapter

dappnode · Dec 12, 2024 · 41fd50d · 41fd50d
1 parent 405fa56
commit 41fd50d
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 7 deletions.
diff --git a/internal/adapters/api/api_adapter.go b/internal/adapters/api/api_adapter.go
@@ -184,13 +184,6 @@ func (h *APIHandler) UpdateTelegramConfig(w http.ResponseWriter, r *http.Request
 		return
 	}
 
-	// Send a test notification after the bot has been updated
-	if err := h.NotifierPort.SendNotification("🔑 Updated telegram configuration successfully"); err != nil {
-		logger.ErrorWithPrefix("API", "Failed to send test notification: %v", err)
-		writeErrorResponse(w, "Failed to send test notification", http.StatusInternalServerError)
-		return
-	}
-
 	w.WriteHeader(http.StatusOK)
 }
 

diff --git a/internal/adapters/notifier/notifier_adapter.go b/internal/adapters/notifier/notifier_adapter.go
@@ -65,7 +65,14 @@ func (tb *TelegramBot) UpdateBotConfig() error {
 
 	tb.Bot = bot
 	tb.UserID = config.UserID
+
+	// Send a test notification after the bot has been updated
+	if err := tb.SendNotification("🔑 Updated telegram configuration successfully"); err != nil {
+		return fmt.Errorf("failed to send test notification: %w", err)
+	}
+
 	logger.InfoWithPrefix(tb.servicePrefix, "Telegram bot configuration updated successfully.")
+
 	return nil
 }