fix: update to npm versions and fix PR comments

.github/workflows/cypress.yml

@@ -48,15 +48,14 @@ jobs:
           VITE_GRAASP_ANALYZER_HOST: ${{ vars.VITE_GRAASP_ANALYZER_HOST }}
           VITE_SHOW_NOTIFICATIONS: ${{ vars.VITE_SHOW_NOTIFICATIONS }}
 
-
       #  use the Cypress GitHub Action to run Cypress tests within the chrome browser
       - name: Cypress run
         uses: cypress-io/github-action@v5
         with:
           install: false
           # we launch the app in preview mode to avoid issues with hmr websockets from vite polluting the mocks
           start: yarn preview:test
-          # browser: chrome
+          browser: chrome
           quiet: true
           config-file: cypress.config.ts
           cache-key: ${{ runner.os }}-modules-${{ hashFiles('**/yarn.lock') }}

package.json

@@ -17,15 +17,15 @@
     "@emotion/react": "11.11.1",
     "@emotion/styled": "11.11.0",
     "@graasp/chatbox": "2.0.1",
-    "@graasp/query-client": "github:graasp/graasp-query-client#websockets",
-    "@graasp/sdk": "github:graasp/graasp-sdk#websockets",
-    "@graasp/translations": "1.19.0",
-    "@graasp/ui": "github:graasp/graasp-ui#521-signout-prop-is-unused-in-itemloginauthorization-component",
+    "@graasp/query-client": "1.11.0",
+    "@graasp/sdk": "1.8.0",
+    "@graasp/translations": "1.19.2",
+    "@graasp/ui": "3.5.4",
     "@mui/icons-material": "5.14.6",
     "@mui/lab": "5.0.0-alpha.141",
     "@mui/material": "5.14.6",
-    "@sentry/react": "7.66.0",
-    "@sentry/tracing": "7.66.0",
+    "@sentry/react": "7.73.0",
+    "@sentry/tracing": "7.73.0",
     "@uppy/core": "3.3.1",
     "@uppy/dashboard": "3.4.2",
     "@uppy/drag-drop": "3.0.2",
@@ -41,7 +41,7 @@
     "date-fns": "2.30.0",
     "filesize": "10.0.12",
     "http-status-codes": "2.2.0",
-    "immutable": "4.3.2",
+    "immutable": "4.3.4",
     "katex": "0.16.8",
     "lodash.groupby": "4.6.0",
     "lodash.partition": "4.6.0",
@@ -102,7 +102,7 @@
     ]
   },
   "devDependencies": {
-    "@commitlint/cli": "17.7.1",
+    "@commitlint/cli": "17.7.2",
     "@commitlint/config-conventional": "17.7.0",
     "@cypress/code-coverage": "3.11.0",
     "@testing-library/jest-dom": "^6.0.0",
@@ -144,8 +144,5 @@
     "vite-plugin-checker": "0.6.2",
     "vite-plugin-istanbul": "5.0.0"
   },
-  "resolutions": {
-    "immutable": "4.3.2"
-  },
   "packageManager": "[email protected]"
 }

src/components/App.tsx

@@ -35,6 +35,7 @@ const App = (): JSX.Element => {
   const { pathname } = useLocation();
   const { data: currentMember, isLoading } = useCurrentUserContext();
 
+  // registers the item updates through websockets
   useItemFeedbackUpdates?.(currentMember?.id);
 
   if (isLoading) {