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

fix cloud build

Author: simbayippy

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

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;