Merge pull request #126 from CS3219-AY2425S1/cloud-fix

fix cloud build
CS3219-AY2425S1 · Nov 3, 2024 · d577860 · d577860
2 parents 2e91c7a + bdf5ebf
commit d577860
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/cloudbuilds/fe.yaml b/cloudbuilds/fe.yaml
@@ -12,6 +12,8 @@ steps:
         "NEXT_PUBLIC_API_GATEWAY_URL=${_NEXT_PUBLIC_API_GATEWAY_URL}",
         "--build-arg",
         "NEXT_PUBLIC_GITHUB_CLIENT_ID=${_NEXT_PUBLIC_GITHUB_CLIENT_ID}",
+        "--build-arg",
+        "NEXT_PUBLIC_SOCK_SERVER_URL=${_NEXT_PUBLIC_SOCK_SERVER_URL}",
         "-f",
         "peerprep-fe/Dockerfile",
         "peerprep-fe",
@@ -52,6 +54,7 @@ steps:
 substitutions:
   _NEXT_PUBLIC_API_GATEWAY_URL: ""
   _NEXT_PUBLIC_GITHUB_CLIENT_ID: ""
+  _NEXT_PUBLIC_SOCK_SERVER_URL: ""
 
 options:
   dynamic_substitutions: true

diff --git a/peerprep-fe/src/app/collaboration/components/CollaborationEditor.tsx b/peerprep-fe/src/app/collaboration/components/CollaborationEditor.tsx
@@ -83,7 +83,7 @@ const CollaborationEditor = ({
       const states = providerRef.current?.awareness.getStates();
       if (states) {
         const newClients = new Map<number, ConnectedClient>();
-        // Build new clients map.
+        // Build new clients map
         // eslint-disable-next-line @typescript-eslint/no-explicit-any
         states.forEach((value: { [x: string]: any }) => {
           const state = value as AwarenessState;