Skip to content

Commit c8dce41

Browse files
committed
fix: upgrade CI/CD workflows to Node.js 22 for navigator API support
- Update both CI and deploy workflows from Node.js 20 to 22 - Node.js 21+ includes navigator global object natively - Fixes certificate.spec.ts failures in CI (ReferenceError: navigator is not defined) - Add PUBLIC_BASE_PATH to deploy workflow for configurable deployment paths - Tests pass locally with Node v22.17.0 and now will pass in CI
1 parent 1281fef commit c8dce41

2 files changed

Lines changed: 3 additions & 2 deletions

File tree

.github/workflows/ci.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ jobs:
1717
- name: Setup Node.js
1818
uses: actions/setup-node@v4
1919
with:
20-
node-version: '20'
20+
node-version: '22'
2121
cache: 'npm'
2222

2323
- name: Install dependencies

.github/workflows/deploy.yml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ jobs:
2525
- name: Setup Node.js
2626
uses: actions/setup-node@v4
2727
with:
28-
node-version: '20'
28+
node-version: '22'
2929
cache: 'npm'
3030

3131
- name: Install dependencies
@@ -34,6 +34,7 @@ jobs:
3434
- name: Build application
3535
run: npm run build
3636
env:
37+
PUBLIC_BASE_PATH: ${{ vars.BASE_PATH || '/open-edu' }}
3738
# Firebase configuration for production
3839
PUBLIC_FIREBASE_API_KEY: ${{ secrets.FIREBASE_API_KEY }}
3940
PUBLIC_FIREBASE_AUTH_DOMAIN: ${{ secrets.FIREBASE_AUTH_DOMAIN }}

0 commit comments

Comments
 (0)