added the mobile-header id back to fix breaking tests; update workflow

aasimsyed · Jul 5, 2024 · 6efcc68 · 6efcc68
1 parent 49f4671
commit 6efcc68
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/.github/workflows/pr_ci_playwright_e2e.yaml b/.github/workflows/pr_ci_playwright_e2e.yaml
@@ -58,11 +58,11 @@ jobs:
           MAX_ATTEMPTS=3
           WAIT_TIME=30
           ATTEMPT=1
-          while [[ $ATTEMPT -le $MAX_ATTEMPTS ]]
-          do
+          while [[ $ATTEMPT -le $MAX_ATTEMPTS ]]; do
             echo "Attempt $ATTEMPT of $MAX_ATTEMPTS"
             yarn wait-on http://localhost:3000 --timeout ${WAIT_TIME}000
-            if [[ $? -eq 0 ]]; then
+            STATUS=$?
+            if [[ $STATUS -eq 0 ]]; then
               echo "Server is ready!"
               break
             else

diff --git a/frontend/components/header/HeaderWebsite.vue b/frontend/components/header/HeaderWebsite.vue
@@ -10,7 +10,7 @@
     }"
   >
     <!-- MARK: Mobile Header -->
-    <div v-if="!aboveMediumBP" class="flex px-4 py-3">
+    <div v-if="!aboveMediumBP" id="#mobile-header" class="flex px-4 py-3">
       <div class="z-0 mx-auto">
         <div
           class="absolute left-0 top-0 z-0 flex h-full w-full items-center justify-center"