Merge pull request #20 from primeroz/fix-notification

Fix notification
primeroz · Dec 30, 2020 · b462797 · b462797
2 parents 83d1926 + 22ab1b4
commit b462797
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 4 deletions.
diff --git a/exchange/kraken.go b/exchange/kraken.go
@@ -118,7 +118,7 @@ func (k *Kraken) Stack(amount float64, fiat string, orderType string, dryRun boo
 
 	var orderId string
 	if dryRun {
-		orderId = "DryRun"
+		orderId = "DRY-RUN"
 	} else {
 		orderId = strings.Join(order.TransactionIds, ",")
 	}

diff --git a/main.go b/main.go
@@ -264,10 +264,13 @@ func main() {
 				strings.Title(action),
 			)
 
-			err := nf.Notify(title, result)
+			// Do not notify if result is not set ( for example if the required args where not specified )
+			if result != "" {
+				err := nf.Notify(title, result)
 
-			if err != nil {
-				return cli.Exit(fmt.Sprintf("Notification Error: %s", err), 1)
+				if err != nil {
+					return cli.Exit(fmt.Sprintf("Notification Error: %s", err), 1)
+				}
 			}
 
 			return nil