Merge pull request #386 from cypherstack/mb-fixes

Mb fixes

lib/models/exchange/majestic_bank/mb_order_status.dart

@@ -24,9 +24,18 @@ class MBOrderStatus extends MBObject {
   final Decimal received;
   final Decimal confirmed;
 
+  Map<String, dynamic> toMap() => {
+        "orderId": orderId,
+        "status": status,
+        "fromCurrency": fromCurrency,
+        "fromAmount": fromAmount,
+        "receiveCurrency": receiveCurrency,
+        "receiveAmount": receiveAmount,
+        "address": address,
+        "received": received,
+        "confirmed": confirmed,
+      };
+
   @override
-  String toString() {
+  String toString() => toMap().toString();
-    // todo: full toString
-    return status;
-  }
 }

lib/pages/exchange_view/trade_details_view.dart

@@ -116,9 +116,16 @@ class _TradeDetailsViewState extends ConsumerState<TradeDetailsView> {
       }
       status = changeNowTransactionStatusFromStringIgnoreCase(statusString);
     } on ArgumentError catch (_) {
-      status = ChangeNowTransactionStatus.Failed;
+      switch (statusString.toLowerCase()) {
-      if (statusString == "Processing payment") {
+        case "funds confirming":
-        status = ChangeNowTransactionStatus.Sending;
+        case "processing payment":
+          return Assets.svg.txExchangePending(context);
+
+        case "completed":
+          return Assets.svg.txExchange(context);
+
+        default:
+          status = ChangeNowTransactionStatus.Failed;
       }
     }
 

lib/pages_desktop_specific/desktop_exchange/desktop_all_trades_view.dart

@@ -290,11 +290,21 @@ class _DesktopTradeRowCardState extends ConsumerState<DesktopTradeRowCard> {
     ChangeNowTransactionStatus? status;
     try {
       if (statusString.toLowerCase().startsWith("waiting")) {
-        statusString = "waiting";
+        statusString = "Waiting";
       }
       status = changeNowTransactionStatusFromStringIgnoreCase(statusString);
     } on ArgumentError catch (_) {
-      status = ChangeNowTransactionStatus.Failed;
+      switch (statusString.toLowerCase()) {
+        case "funds confirming":
+        case "processing payment":
+          return Assets.svg.txExchangePending(context);
+
+        case "completed":
+          return Assets.svg.txExchange(context);
+
+        default:
+          status = ChangeNowTransactionStatus.Failed;
+      }
     }
 
     switch (status) {

lib/services/exchange/majestic_bank/majestic_bank_api.dart

@@ -349,8 +349,10 @@ class MajesticBankAPI {
 
       return ExchangeResponse(value: status);
     } catch (e, s) {
-      Logging.instance
+      Logging.instance.log(
-          .log("createOrder exception: $e\n$s", level: LogLevel.Error);
+        "trackOrder exception when trying to parse $json: $e\n$s",
+        level: LogLevel.Error,
+      );
       return ExchangeResponse(
         exception: ExchangeException(
           e.toString(),

lib/utilities/theme/stack_colors.dart

@@ -1603,6 +1603,7 @@ class StackColors extends ThemeExtension<StackColors> {
         return const Color(0xFFD3A90F);
       case "Finished":
       case "finished":
+      case "Completed":
         return accentColorGreen;
       case "Failed":
       case "failed":

lib/widgets/trade_card.dart

@@ -25,11 +25,21 @@ class TradeCard extends ConsumerWidget {
     ChangeNowTransactionStatus? status;
     try {
       if (statusString.toLowerCase().startsWith("waiting")) {
-        statusString = "waiting";
+        statusString = "Waiting";
       }
       status = changeNowTransactionStatusFromStringIgnoreCase(statusString);
     } on ArgumentError catch (_) {
-      status = ChangeNowTransactionStatus.Failed;
+      switch (statusString.toLowerCase()) {
+        case "funds confirming":
+        case "processing payment":
+          return Assets.svg.txExchangePending(context);
+
+        case "completed":
+          return Assets.svg.txExchange(context);
+
+        default:
+          status = ChangeNowTransactionStatus.Failed;
+      }
     }
 
     switch (status) {