Merge branch 'master' into feature/listen-to-ports

Author: odeimaiz

Makefile

@@ -128,7 +128,8 @@ help: ## help on rule's targets
 
 
 test_python_version: ## Check Python version, throw error if compilation would fail with the installed version
-	@uv run ./scripts/test_python_version.py
+	# Checking python version
+	@.venv/bin/python ./scripts/test_python_version.py
 
 
 ## DOCKER BUILD -------------------------------
@@ -482,7 +483,7 @@ push-version: tag-version
 
 .venv: .check-uv-installed
 	@uv venv $@
-	## upgrading tools to latest version in $(shell python3 --version)
+	@echo "# upgrading tools to latest version in" && $@/bin/python --version
 	@uv pip --quiet install --upgrade \
 		pip~=24.0 \
 		wheel \

packages/models-library/src/models_library/api_schemas_dynamic_sidecar/ports.py

@@ -11,14 +11,14 @@ class OutputStatus(StrAutoEnum):
     UPLOAD_STARTED = auto()
     UPLOAD_WAS_ABORTED = auto()
     UPLOAD_FINISHED_SUCCESSFULLY = auto()
-    UPLOAD_FINISHED_WITH_ERRROR = auto()
+    UPLOAD_FINISHED_WITH_ERROR = auto()
 
 
 class InputStatus(StrAutoEnum):
     DOWNLOAD_STARTED = auto()
     DOWNLOAD_WAS_ABORTED = auto()
     DOWNLOAD_FINISHED_SUCCESSFULLY = auto()
-    DOWNLOAD_FINISHED_WITH_ERRROR = auto()
+    DOWNLOAD_FINISHED_WITH_ERROR = auto()
 
 
 class _PortStatusCommon(BaseModel):

services/dynamic-sidecar/src/simcore_service_dynamic_sidecar/modules/notifications/_notifications_ports.py

@@ -52,7 +52,7 @@ async def send_output_port_upload_finished_with_error(
         self, port_key: ServicePortKey
     ) -> None:
         await self._send_output_port_status(
-            port_key, OutputStatus.UPLOAD_FINISHED_WITH_ERRROR
+            port_key, OutputStatus.UPLOAD_FINISHED_WITH_ERROR
         )
 
     async def send_input_port_download_started(self, port_key: ServicePortKey) -> None:
@@ -74,5 +74,5 @@ async def send_input_port_download_finished_with_error(
         self, port_key: ServicePortKey
     ) -> None:
         await self._send_input_port_status(
-            port_key, InputStatus.DOWNLOAD_FINISHED_WITH_ERRROR
+            port_key, InputStatus.DOWNLOAD_FINISHED_WITH_ERROR
         )

services/dynamic-sidecar/tests/unit/test_modules_notifier.py

@@ -289,7 +289,7 @@ async def test_notifier_send_input_port_status(
                 await port_notifier.send_input_port_download_finished_succesfully(
                     port_key
                 )
-            case InputStatus.DOWNLOAD_FINISHED_WITH_ERRROR:
+            case InputStatus.DOWNLOAD_FINISHED_WITH_ERROR:
                 await port_notifier.send_input_port_download_finished_with_error(
                     port_key
                 )
@@ -378,7 +378,7 @@ async def test_notifier_send_output_port_status(
                 await port_notifier.send_output_port_upload_finished_successfully(
                     port_key
                 )
-            case OutputStatus.UPLOAD_FINISHED_WITH_ERRROR:
+            case OutputStatus.UPLOAD_FINISHED_WITH_ERROR:
                 await port_notifier.send_output_port_upload_finished_with_error(
                     port_key
                 )

services/static-webserver/client/source/class/osparc/dashboard/ContextBreadcrumbs.js

@@ -52,6 +52,10 @@ qx.Class.define("osparc.dashboard.ContextBreadcrumbs", {
     __rebuild: function() {
       this._removeAll();
 
+      if (this.getCurrentWorkspaceId() === -2) {
+        return;
+      }
+
       if (this.getCurrentFolderId()) {
         const currentFolder = osparc.store.Folders.getInstance().getFolder(this.getCurrentFolderId());
         this.__createUpstreamButtons(currentFolder);

services/static-webserver/client/source/class/osparc/dashboard/Dashboard.js

@@ -138,7 +138,8 @@ qx.Class.define("osparc.dashboard.Dashboard", {
         });
         const tabButton = tabPage.getChildControl("button");
         tabButton.set({
-          minWidth: 50
+          minWidth: 50,
+          maxHeight: 36,
         });
         tabButton.ttt = label;
         tabButton.getChildControl("label").set({

services/static-webserver/client/source/class/osparc/dashboard/FolderButtonBase.js

@@ -112,12 +112,10 @@ qx.Class.define("osparc.dashboard.FolderButtonBase", {
     },
 
     _shouldApplyFilter: function(data) {
-      console.log("_shouldApplyFilter", data);
       return false;
     },
 
     _shouldReactToFilter: function(data) {
-      console.log("_shouldReactToFilter", data);
       return false;
     }
   },

services/static-webserver/client/source/class/osparc/dashboard/ResourceBrowserBase.js

@@ -42,15 +42,15 @@ qx.Class.define("osparc.dashboard.ResourceBrowserBase", {
     const mainLayoutWithSideSpacers = new qx.ui.container.Composite(new qx.ui.layout.HBox(spacing))
     this._addToMainLayout(mainLayoutWithSideSpacers);
 
-    this.__leftFilters = new qx.ui.container.Composite(new qx.ui.layout.VBox(10)).set({
+    this.__leftFilters = new qx.ui.container.Composite(new qx.ui.layout.VBox(15)).set({
       width: leftColumnWidth
     });
     mainLayoutWithSideSpacers.add(this.__leftFilters);
 
-    this.__centerLayout = new qx.ui.container.Composite(new qx.ui.layout.VBox(10));
+    this.__centerLayout = new qx.ui.container.Composite(new qx.ui.layout.VBox(15));
     mainLayoutWithSideSpacers.add(this.__centerLayout);
 
-    const rightColum = new qx.ui.container.Composite(new qx.ui.layout.VBox(10));
+    const rightColum = new qx.ui.container.Composite(new qx.ui.layout.VBox());
     mainLayoutWithSideSpacers.add(rightColum, {
       flex: 1
     });
@@ -236,6 +236,7 @@ qx.Class.define("osparc.dashboard.ResourceBrowserBase", {
       });
       const textField = searchBarFilter.getChildControl("text-field");
       osparc.utils.Utils.setIdToWidget(textField, "searchBarFilter-textField-"+this._resourceType);
+
       this._addToLayout(searchBarFilter);
     },
 
@@ -356,6 +357,15 @@ qx.Class.define("osparc.dashboard.ResourceBrowserBase", {
         radioGroup.add(btn);
       });
 
+      if (this._resourceType === "study") {
+        const viewMode = osparc.utils.Utils.localCache.getLocalStorageItem("studiesViewMode");
+        if (viewMode) {
+          if (viewMode === "list") {
+            radioGroup.setSelection([listBtn]);
+          }
+        }
+      }
+
       this._toolbar.add(viewModeLayout);
     },
 

services/static-webserver/client/source/class/osparc/dashboard/ResourceContainerManager.js

@@ -32,10 +32,6 @@ qx.Class.define("osparc.dashboard.ResourceContainerManager", {
     this.__resourcesList = [];
     this.__groupedContainersList = [];
 
-    const containerHeader = this.__containerHeader = new osparc.dashboard.ContextBreadcrumbs();
-    this._add(containerHeader);
-    containerHeader.setVisibility(osparc.utils.DisabledPlugins.isFoldersEnabled() ? "visible" : "excluded");
-
 
     const workspacesContainer = this.__workspacesContainer = new osparc.dashboard.ToggleButtonContainer();
     workspacesContainer.setVisibility(osparc.utils.DisabledPlugins.isFoldersEnabled() ? "visible" : "excluded");
@@ -64,7 +60,7 @@ qx.Class.define("osparc.dashboard.ResourceContainerManager", {
       init: "grid",
       nullable: false,
       event: "changeMode",
-      apply: "reloadCards"
+      apply: "__reloadCards"
     },
 
     groupBy: {
@@ -117,8 +113,6 @@ qx.Class.define("osparc.dashboard.ResourceContainerManager", {
     __workspacesList: null,
     __resourcesList: null,
     __groupedContainersList: null,
-    __foldersLayout: null,
-    __containerHeader: null,
     __foldersContainer: null,
     __workspacesContainer: null,
     __nonGroupedContainer: null,
@@ -163,10 +157,6 @@ qx.Class.define("osparc.dashboard.ResourceContainerManager", {
       }
     },
 
-    getContainerHeader: function() {
-      return this.__containerHeader;
-    },
-
     getFlatList: function() {
       return this.__nonGroupedContainer;
     },
@@ -233,7 +223,10 @@ qx.Class.define("osparc.dashboard.ResourceContainerManager", {
         position: "bottom-right"
       });
       card.setMenu(menu);
-      card.subscribeToFilterGroup("searchBarFilter");
+      if (resourceData.type !== "study") {
+        // the backend will do the projects:search
+        card.subscribeToFilterGroup("searchBarFilter");
+      }
 
       [
         "updateStudy",
@@ -284,6 +277,10 @@ qx.Class.define("osparc.dashboard.ResourceContainerManager", {
       this._removeAll();
     },
 
+    __reloadCards: function(mode) {
+      this.reloadCards();
+    },
+
     __addFoldersContainer: function() {
       // add foldersContainer dynamically
       [
@@ -356,7 +353,6 @@ qx.Class.define("osparc.dashboard.ResourceContainerManager", {
 
     reloadWorkspaces: function() {
       this.__cleanAll();
-      this._add(this.__containerHeader);
       this._add(this.__workspacesContainer);
       let workspacesCards = [];
       this.__workspacesList.forEach(workspaceData => workspacesCards.push(this.__workspaceToCard(workspaceData)));
@@ -375,7 +371,6 @@ qx.Class.define("osparc.dashboard.ResourceContainerManager", {
 
     __createWorkspaceCard: function(workspace) {
       const card = new osparc.dashboard.WorkspaceButtonItem(workspace);
-      card.subscribeToFilterGroup("searchBarFilter");
       [
         "workspaceSelected",
         "workspaceUpdated",
@@ -411,7 +406,6 @@ qx.Class.define("osparc.dashboard.ResourceContainerManager", {
 
     __createFolderCard: function(folder) {
       const card = new osparc.dashboard.FolderButtonItem(folder);
-      card.subscribeToFilterGroup("searchBarFilter");
       [
         "folderSelected",
         "folderUpdated",

services/static-webserver/client/source/class/osparc/dashboard/ResourceDetails.js

@@ -159,7 +159,11 @@ qx.Class.define("osparc.dashboard.ResourceDetails", {
             this.__openResource();
           }
         })
-        .catch(() => this.__openButton.setFetching(false));
+        .catch(err => {
+          console.error(err);
+          osparc.FlashMessenger.logAs(err.message, "ERROR");
+          this.__openButton.setFetching(false);
+        });
     },
 
     __confirmUpdate: function() {