Revert "[hotfix] don't update transaction status, of cancelled or successful trades, patch suspected bug of orders in status 13 being set to 99 from bad request response"

This reverts commit 1a658e2.

Author: tmwclaxton

app/Jobs/UpdateTransactions.php

@@ -33,7 +33,7 @@ public function __construct()
     public function handle(): void
     {
         // update all current transactions where status != 14, 12, 17, 18, 99
-        $transactions = Transaction::whereNotIn('status', [4, 5, 14, 12, 14, 17, 18, 99])->get();
+        $transactions = Transaction::whereNotIn('status', [5, 14, 12, 17, 18, 99])->get();
         foreach ($transactions as $transaction) {
             $offer = $transaction->offer;
             $robosats = new Robosats();

app/WorkerClasses/Robosats.php

@@ -866,11 +866,14 @@ public function updateTransactionStatus($offer) {
             }
 
 
-            // I think robosats returns a bad_request response before trade hits a 14 completed status i.e. sending sats to the buyer, and we should just set expires_at to now rather than setting status to 99
             if ($offer->status < 14) {
+                $offer->status_message = $response['bad_request'];
+                $offer->status = 99;
                 // set expires at to now
                 $offer->expires_at = date('Y-m-d H:i:s');
                 $offer->save();
+                $transaction->status_message = $response['bad_request'];
+                $transaction->status = 99;
                 $transaction->save();
             } else {
 //                $slackService = new SlackService();