Merge pull request #120 from rerades/env-var-ga

Refactor CI workflow and Google Analytics component

Author: area73

.github/workflows/ci.yml

@@ -57,6 +57,8 @@ jobs:
   E2E-Tests:
     name: E2E-Tests
     runs-on: ubuntu-latest
+    # env:
+    #  PUBLIC_GA_MEASUREMENT_ID: ${{ vars.PUBLIC_GA_MEASUREMENT_ID }}
     steps:
       - uses: actions/checkout@v4
       - uses: pnpm/action-setup@v4
@@ -83,7 +85,7 @@ jobs:
       PORT: 4321
       HOST: 0.0.0.0
       BASE_URL: http://127.0.0.1:4321
-      PUBLIC_GA_MEASUREMENT_ID: ${{ vars.PUBLIC_GA_MEASUREMENT_ID }}
+      # PUBLIC_GA_MEASUREMENT_ID: ${{ vars.PUBLIC_GA_MEASUREMENT_ID }}
     steps:
       - uses: actions/checkout@v4
       - uses: pnpm/action-setup@v4

src/components/GoogleAnalytics.astro

@@ -10,7 +10,6 @@ const gaMeasurementId = import.meta.env.PUBLIC_GA_MEASUREMENT_ID;
     <>
       {/* Google Analytics 4 - Optimized non-blocking implementation */}
       {/* Script loads asynchronously with low priority to not compete with critical resources */}
-      {/* @ts-expect-error - fetchpriority is valid HTML but not yet in TypeScript definitions */}
       <script
         async
         src={`https://www.googletagmanager.com/gtag/js?id=${gaMeasurementId}`}