Merge branch 'dev' into firoorg-dev

KomodoPlatform · Oct 28, 2024 · 34a0307 · 34a0307
2 parents 17394b4 + 0e2e158
commit 34a0307
Show file tree

Hide file tree

Showing 20 changed files with 67 additions and 48 deletions.
diff --git a/.github/ISSUE_TEMPLATE/bug_report.md b/.github/ISSUE_TEMPLATE/bug_report.md
@@ -26,7 +26,7 @@ Please describe what you expected to happen.
 **Operating Environment(s):**
  - OS: [e.g. Windows/OSX/Linux. If Linux, include distro. ]
  - OS version: [e.g. 7/10/11, 10.13/10.15, 18.04/20.04 ]
- - Komodo Wallet Desktop Version: [e.g. 0.8.0]
+ - Komodo Wallet Desktop Version: [e.g. 0.8.1]
  - Build branch: [e.g. master/dev] 
 
 

diff --git a/.github/workflows/firodex-desktop-cd.yml b/.github/workflows/firodex-desktop-cd.yml
@@ -9,7 +9,7 @@ env:
   DEX_PROJECT_NAME: "firodex-desktop"
   DEX_DISPLAY_NAME: "FiroDex Desktop"
   DEX_COMPANY: "Firo Core Team"
-  DEX_VERSION: "0.8.0"
+  DEX_VERSION: "0.8.1"
   DEX_WEBSITE: "https://firo.org/"
 
 jobs:
@@ -127,7 +127,7 @@ jobs:
           zip AtomicDEX-Pro-Installer.zip ./installer/AtomicDEX-Pro-Installer
 
       - name: Upload artifacts (Linux)
-        uses: actions/upload-artifact@v1
+        uses: actions/upload-artifact@v4
         with:
           name: AtomicDEX-Pro-Installer-linux-release
           path: ./ci_tools_atomic_dex/AtomicDEX-Pro-Installer.zip
@@ -239,7 +239,7 @@ jobs:
           zip AtomicDEX-Pro-Installer.zip ./installer/AtomicDEX-Pro-Installer
 
       - name: Upload artifacts (Linux/Debug)
-        uses: actions/upload-artifact@v1
+        uses: actions/upload-artifact@v4
         with:
           name: AtomicDEX-Pro-Installer-linux-debug
           path: ./ci_tools_atomic_dex/AtomicDEX-Pro-Installer.zip
@@ -303,7 +303,7 @@ jobs:
           ls bundle-Release/atomicDEX-Pro.dmg
 
       - name: Upload artifacts (MacOS/Release)
-        uses: actions/upload-artifact@v1
+        uses: actions/upload-artifact@v4
         with:
           name: dexpro-mac-release.dmg
           path: ./ci_tools_atomic_dex/bundle-Release/atomicDEX-Pro.dmg
@@ -367,7 +367,7 @@ jobs:
           ls bundle-Debug/atomicDEX-Pro.dmg
 
       - name: Upload artifacts (MacOS/Debug)
-        uses: actions/upload-artifact@v1
+        uses: actions/upload-artifact@v4
         with:
           name: dexpro-mac-debug.dmg
           path: ./ci_tools_atomic_dex/bundle-Debug/atomicDEX-Pro.dmg
@@ -445,7 +445,7 @@ jobs:
           ls bundle-Release/bundle.zip
 
       - name: Upload artifacts (Win/Release)
-        uses: actions/upload-artifact@v1
+        uses: actions/upload-artifact@v4
         with:
           name: dexpro-win-release
           path: ./ci_tools_atomic_dex/bundle-Release/bundle.zip
@@ -523,7 +523,7 @@ jobs:
           ls bundle-Debug/bundle.zip
 
       - name: Upload artifacts (Windows/Debug)
-        uses: actions/upload-artifact@v1
+        uses: actions/upload-artifact@v4
         with:
           name: dexpro-win-debug
           path: ./ci_tools_atomic_dex/bundle-Debug/bundle.zip

diff --git a/.github/workflows/firodex-desktop-ci.yml b/.github/workflows/firodex-desktop-ci.yml
@@ -18,7 +18,7 @@ env:
   DEX_PROJECT_NAME: "firodex"
   DEX_DISPLAY_NAME: "Firo Dex"
   DEX_COMPANY: "Firo"
-  DEX_VERSION: "0.8.0"
+  DEX_VERSION: "0.8.1"
   DEX_WEBSITE: "https://firo.org/"
   VCPKG_DEFAULT_BINARY_CACHE: ${{ github.workspace }}/b/vcpkg_cache
   VCPKG_BINARY_SOURCES: clear;x-gha,readwrite
@@ -239,7 +239,7 @@ jobs:
           echo $HOME/sdk
           ls $HOME/sdk
           export MAC_SIGN_IDENTITY="${{ secrets.MAC_SIGN_IDENTITY }}"
-          export INSTALLER_MAC_SIGN_IDENTITY="${{ secrets.INSTALLER_MAC_SIGN_IDENTITY }}"
+          export INSTALLER_MAC_SIGN_IDENTITY="${{ secrets.MAC_SIGN_IDENTITY }}"
           export APPLE_ATOMICDEX_PASSWORD="${{ secrets.APPLE_ATOMICDEX_PASSWORD }}"
           export APPLE_ID="${{ secrets.APPLE_ID }}"
           export ASC_PUBLIC_ID="${{ secrets.ASC_PUBLIC_ID }}"
@@ -293,23 +293,23 @@ jobs:
 
       - name: Upload bundle artifact (Linux ZSTD)
         if: runner.os == 'Linux'
-        uses: actions/upload-artifact@v3
+        uses: actions/upload-artifact@v4
         with:
           name: ${{ env.artifact_name_zstd }}
           path: ./bundled/linux/${{ env.target_name_zstd }}
           retention-days: 7
 
       - name: Upload bundle artifact (Linux ZIP)
         if: runner.os == 'Linux'
-        uses: actions/upload-artifact@v3
+        uses: actions/upload-artifact@v4
         with:
           name: ${{ env.artifact_name_zip }}
           path: ./bundled/linux/${{ env.target_name_zip }}
           retention-days: 7
 
       - name: Upload bundle artifact (Linux AppImage)
         if: runner.os == 'Linux'
-        uses: actions/upload-artifact@v3
+        uses: actions/upload-artifact@v4
         with:
           name: ${{ env.artifact_name_appimage }}
           path: ./bundled/linux/${{ env.target_name_appimage }}
@@ -323,15 +323,15 @@ jobs:
 
       - name: Upload artifacts (MacOS dmg)
         if: runner.os == 'macOS'
-        uses: actions/upload-artifact@v3
+        uses: actions/upload-artifact@v4
         with:
           name: ${{ env.artifact_name_dmg }}
           path: ./bundled/osx/${{ env.DEX_PROJECT_NAME }}.dmg
           retention-days: 7
 
       - name: Upload artifacts (MacOS installer)
         if: runner.os == 'macOS'
-        uses: actions/upload-artifact@v3
+        uses: actions/upload-artifact@v4
         with:
           name: ${{ env.artifact_name_installer }}
           path: ./bundled/osx/${{ env.DEX_PROJECT_NAME }}_installer.7z
@@ -346,15 +346,15 @@ jobs:
 
       - name: Upload artifacts (Windows zip)
         if: runner.os == 'Windows'
-        uses: actions/upload-artifact@v3
+        uses: actions/upload-artifact@v4
         with:
           name: ${{ env.artifact_name_zip }}
           path: ./bundled/windows/${{ env.DEX_PROJECT_NAME }}.zip
           retention-days: 7
 
       - name: Upload artifacts (Windows installer)
         if: runner.os == 'Windows'
-        uses: actions/upload-artifact@v3
+        uses: actions/upload-artifact@v4
         with:
           name: ${{ env.artifact_name_installer }}
           path: ./bundled/windows/${{ env.DEX_PROJECT_NAME }}_installer.exe
@@ -387,7 +387,7 @@ jobs:
         if: runner.os == 'macOS' &&  'firoorg/FiroDEX-Desktop' == github.repository
         run: |
           export MAC_SIGN_IDENTITY="${{ secrets.MAC_SIGN_IDENTITY }}"
-          export INSTALLER_MAC_SIGN_IDENTITY="${{ secrets.INSTALLER_MAC_SIGN_IDENTITY }}"
+          export INSTALLER_MAC_SIGN_IDENTITY="${{ secrets.MAC_SIGN_IDENTITY }}"
           export APPLE_ATOMICDEX_PASSWORD="${{ secrets.APPLE_ATOMICDEX_PASSWORD }}"
           export APPLE_ID="${{ secrets.APPLE_ID }}"
           export ASC_PUBLIC_ID="${{ secrets.ASC_PUBLIC_ID }}"

diff --git a/CMakeLists.txt b/CMakeLists.txt
@@ -11,7 +11,7 @@ include(vcpkg_prerequisites)
 include(qt_prerequisites)
 include(cfg_hash)
 
-project(${DEX_PROJECT_NAME} LANGUAGES CXX VERSION 0.8.0)
+project(${DEX_PROJECT_NAME} LANGUAGES CXX VERSION 0.8.1)
 message(STATUS "${PROJECT_NAME} is version ${PROJECT_VERSION}")
 
 include(cmake_default_options)
@@ -60,13 +60,13 @@ endif ()
 ##! We fetch our dependencies
 if (APPLE)
     FetchContent_Declare(kdf
-            URL https://github.com/KomodoPlatform/komodo-defi-framework/releases/download/v2.1.0-beta/mm2-35e923949-Darwin-Release.zip)
+            URL https://sdk.devbuilds.komodo.earth/dev/mm2_b35a818-mac-x86-64.zip)
 elseif (UNIX AND NOT APPLE)
     FetchContent_Declare(kdf
-            URL https://github.com/KomodoPlatform/komodo-defi-framework/releases/download/v2.1.0-beta/mm2-35e923949-Linux-Release.zip)
+            URL https://sdk.devbuilds.komodo.earth/dev/mm2_b35a818-linux-x86-64.zip)
 else ()
     FetchContent_Declare(kdf
-            URL https://github.com/KomodoPlatform/komodo-defi-framework/releases/download/v2.1.0-beta/mm2-35e923949-Win64.zip)
+            URL https://sdk.devbuilds.komodo.earth/dev/mm2_b35a818-win-x86-64.zip)
 endif ()
 
 #FetchContent_Declare(qmaterial URL https://github.com/KomodoPlatform/Qaterial/archive/last-clang-working-2.zip)

diff --git a/ci_tools_atomic_dex/ci_scripts/osx_script.sh b/ci_tools_atomic_dex/ci_scripts/osx_script.sh
@@ -12,9 +12,12 @@ brew install autoconf \
             gnu-sed \
             coreutils \
             libtool \
-            llvm \
             gnu-getopt
 
+brew unlink [email protected]
+brew install llvm
+brew link --overwrite [email protected]
+
 pip3 install yq
 export CC=clang
 export CXX=clang++

diff --git a/cmake/project.metadata.cmake b/cmake/project.metadata.cmake
@@ -6,7 +6,7 @@ set(DEX_PROJECT_NAME "firodex")
 set(DEX_DISPLAY_NAME "Firo Dex")
 set(DEX_MAINTENANCE_TOOL_NAME "Firo Dex Maintenance Tool")
 set(DEX_COMPANY "Firo")
-set(DEX_VERSION "0.8.0")
+set(DEX_VERSION "0.8.1")
 set(DEX_WEBSITE "https://firo.org/")
 set(DEX_SUPPORT_PAGE "https://firo.org/guide/")
 set(DEX_DISCORD "https://discord.com/invite/TGZPRbRT3Y")

diff --git a/src/core/atomicdex/api/kdf/address_format.hpp b/src/core/atomicdex/api/kdf/address_format.hpp
@@ -16,6 +16,7 @@
 
 #pragma once
 
+#include <optional>
 #include <string>
 #include <nlohmann/json_fwd.hpp>
 

diff --git a/src/core/atomicdex/api/kdf/kdf.client.cpp b/src/core/atomicdex/api/kdf/kdf.client.cpp
@@ -205,20 +205,20 @@ namespace atomic_dex::kdf
         auto http_request = make_request<Rpc>(request);
         generate_client()
             .request(http_request, m_token_source.get_token())
-            .template then([on_rpc_processed, request](const web::http::http_response& resp)
-                           {
-                               try
-                               {
-                                   auto rpc = process_rpc_answer<Rpc>(resp);
-                                   rpc.request = request;
-                                   on_rpc_processed(rpc);
-                               }
-                               catch (const std::exception& ex)
-                               {
-                                   // SPDLOG_DEBUG("process_rpc_answer rpc.result: {}", rpc.raw_result);
-                                   SPDLOG_ERROR(ex.what());
-                               }
-                           });
+            .then([on_rpc_processed, request](const web::http::http_response& resp)
+            {
+                try
+                {
+                    auto rpc = process_rpc_answer<Rpc>(resp);
+                    rpc.request = request;
+                    on_rpc_processed(rpc);
+                }
+                catch (const std::exception& ex)
+                {
+                    // SPDLOG_DEBUG("process_rpc_answer rpc.result: {}", rpc.raw_result);
+                    SPDLOG_ERROR(ex.what());
+                }
+            });
     }
 
     void

diff --git a/src/core/atomicdex/api/kdf/kdf.hpp b/src/core/atomicdex/api/kdf/kdf.hpp
@@ -22,6 +22,7 @@
 //! Deps
 #include <antara/gaming/ecs/system.manager.hpp>
 #include <nlohmann/json.hpp>
+#include <optional>
 
 //! Project Headers
 #include "atomicdex/config/coins.cfg.hpp"

diff --git a/src/core/atomicdex/api/kdf/rpc_v2/rpc2.enable_erc20.hpp b/src/core/atomicdex/api/kdf/rpc_v2/rpc2.enable_erc20.hpp
@@ -17,6 +17,7 @@
 #pragma once
 
 #include <string>
+#include <optional>
 
 #include <nlohmann/json_fwd.hpp> //> nlohmann::json
 #include "atomicdex/api/kdf/rpc.hpp"

diff --git a/src/core/atomicdex/config/raw.kdf.coins.cfg.hpp b/src/core/atomicdex/config/raw.kdf.coins.cfg.hpp
@@ -7,6 +7,7 @@
 //! Deps
 #include <antara/gaming/core/real.path.hpp>
 #include <nlohmann/json.hpp>
+#include <optional>
 
 //! Project
 #include "atomicdex/api/kdf/kdf.constants.hpp"

diff --git a/src/core/atomicdex/models/qt.orderbook.model.cpp b/src/core/atomicdex/models/qt.orderbook.model.cpp
@@ -524,7 +524,7 @@ namespace atomic_dex
     void
     orderbook_model::refresh_orderbook_model_data(const t_orders_contents& orderbook, bool is_bestorders)
     {
-        // SPDLOG_DEBUG("[orderbook_model::refresh_orderbook_model_data], is_bestorders: {}", is_bestorders);
+        SPDLOG_DEBUG("[orderbook_model::refresh_orderbook_model_data], is_bestorders: {}", is_bestorders);
         auto refresh_functor = [this](const std::vector<kdf::order_contents>& contents)
         {
             for (auto&& order: contents)

diff --git a/src/core/atomicdex/models/qt.portfolio.model.cpp b/src/core/atomicdex/models/qt.portfolio.model.cpp
@@ -131,7 +131,9 @@ namespace atomic_dex
                 QJsonObject status = nlohmann_json_object_to_qt_json_object(coin_info.activation_status);
                 update_value(ActivationStatus, status, idx, *this);
                 SPDLOG_DEBUG("updated activation status of: {}", ticker);
+                return true;
             }
+            return false;
         }
     }
 

diff --git a/src/core/atomicdex/pages/qt.portfolio.page.cpp b/src/core/atomicdex/pages/qt.portfolio.page.cpp
@@ -163,12 +163,14 @@ namespace atomic_dex
     void
     portfolio_page::set_chart_category(WalletChartsCategories category)
     {
+        SPDLOG_INFO("new m_current_chart_category: {}", m_current_chart_category);
+        SPDLOG_INFO("qint32(category): {}", qint32(category));
         SPDLOG_INFO("new chart category: {}", QMetaEnum::fromType<WalletChartsCategories>().valueToKey(category));
         if (m_current_chart_category != category)
         {
             m_current_chart_category = category;
             QSettings& settings      = entity_registry_.ctx<QSettings>();
-            settings.setValue("WalletChartsCategory", qint32(m_current_chart_category));
+            settings.setValue("WalletChartsCategory", qint32(category));
             if (m_system_manager.get_system<kdf_service>().is_kdf_running() && m_system_manager.has_system<coingecko_wallet_charts_service>())
             {
                 m_system_manager.get_system<coingecko_wallet_charts_service>().manual_refresh("set_chart_category");

diff --git a/src/core/atomicdex/pages/qt.portfolio.page.hpp b/src/core/atomicdex/pages/qt.portfolio.page.hpp
@@ -51,7 +51,7 @@ namespace atomic_dex
         global_coins_cfg_model*  m_global_cfg_mdl;
         QString                  m_current_balance_all{"0"};
         QString                  m_main_current_balance_all{"0"};
-        WalletChartsCategories   m_current_chart_category;
+        WalletChartsCategories   m_current_chart_category{WalletChartsCategories::OneMonth};
 
       public:
         //! Constructor

diff --git a/src/core/atomicdex/pages/qt.settings.page.cpp b/src/core/atomicdex/pages/qt.settings.page.cpp
@@ -315,7 +315,7 @@ namespace atomic_dex
             {
                 SPDLOG_WARN("Cannot change currency to {} for reason: {}", current_currency.toStdString(), reason);
                 // Try next in line
-                int8_t selected_idx = utils::get_index_str(m_config.possible_currencies, current_currency.toStdString());
+                unsigned long selected_idx = utils::get_index_str(m_config.possible_currencies, current_currency.toStdString());
                 if (selected_idx < m_config.possible_currencies.size() - 1)
                 {
                     set_current_currency(QString::fromStdString(m_config.possible_currencies[selected_idx + 1]));

diff --git a/src/core/atomicdex/pages/qt.trading.page.cpp b/src/core/atomicdex/pages/qt.trading.page.cpp
@@ -810,7 +810,7 @@ namespace atomic_dex
             SPDLOG_WARN("KDF service not available, required to clear forms - skipping");
             return;
         }
-        // SPDLOG_DEBUG("clearing forms : {}", from.toStdString());
+        SPDLOG_DEBUG("clearing forms : {}", from.toStdString());
 
         if (m_preferred_order.has_value() && m_current_trading_mode == TradingModeGadget::Simple &&
             m_selected_order_status == SelectedOrderGadget::OrderNotExistingAnymore)

diff --git a/src/core/atomicdex/services/price/global.provider.cpp b/src/core/atomicdex/services/price/global.provider.cpp
@@ -99,6 +99,14 @@ namespace atomic_dex
     global_price_service::refresh_other_coins_rates(
         const std::string& quote_id, const std::string& ticker, bool with_update_providers, std::atomic_uint16_t nb_try)
     {
+        SPDLOG_DEBUG("refresh_other_coins_rates: {} - {} - {} - {}", quote_id, ticker, with_update_providers, nb_try);
+        if (nb_try > 3)
+        {
+            SPDLOG_ERROR("Failed to fetch rates for ticker after 3 tries: {}", ticker);
+            this->m_coin_rate_providers[ticker] = "0.00";
+            return;
+        }
+
         t_float_50 price = safe_float(get_rate_conversion("USD", ticker, true));
         if (price <= 0)
         {

diff --git a/src/core/atomicdex/version/version.hpp b/src/core/atomicdex/version/version.hpp
@@ -21,24 +21,24 @@ namespace atomic_dex
     constexpr const char*
     get_version()
     {
-        return "0.8.0-beta";
+        return "0.8.1-beta";
     }
 
     constexpr int
     get_num_version() noexcept
     {
-        return 80;
+        return 81;
     }
 
     constexpr const char*
     get_raw_version()
     {
-        return "0.8.0";
+        return "0.8.1";
     }
 
     constexpr const char*
     get_precedent_raw_version()
     {
-        return "0.7.2";
+        return "0.8.0";
     }
 } // namespace atomic_dex
diff --git a/vcpkg.json b/vcpkg.json
@@ -1,6 +1,6 @@
 {
   "name": "firodex-wallet",
-  "version-string": "0.8.0",
+  "version-string": "0.8.1",
   "dependencies": [
     "entt",
     "boost-multiprecision",