fix: 如果未设置统计的环境变量，则不加载相应的代码 #65

meetqy · Mar 25, 2024 · dfc67cc · dfc67cc
1 parent 5638499
commit dfc67cc
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 4 deletions.
diff --git a/src/app/metrics/google-analytics.tsx b/src/app/metrics/google-analytics.tsx
@@ -2,7 +2,9 @@
 
 import Script from "next/script";
 
-const GoogleAnalytics = ({ id }: { id: string }) => {
+const GoogleAnalytics = ({ id }: { id?: string }) => {
+  if (!id) return null;
+
   return (
     <>
       <Script

diff --git a/src/app/metrics/microsoft-clarity.tsx b/src/app/metrics/microsoft-clarity.tsx
@@ -2,7 +2,9 @@
 
 import Script from "next/script";
 
-const MicrosoftClarity = ({ id }: { id: string }) => {
+const MicrosoftClarity = ({ id }: { id?: string }) => {
+  if (!id) return null;
+
   return (
     <Script
       id="microsoft-clarity-init"

diff --git a/src/app/root.tsx b/src/app/root.tsx
@@ -43,8 +43,8 @@ export default function Root({
           </ThemeProvider>
         </TRPCReactProvider>
 
-        <GoogleAnalytics id={process.env.NEXT_PUBLIC_GA_ID!} />
-        <MicrosoftClarity id={process.env.NEXT_PUBLIC_MC_ID!} />
+        <GoogleAnalytics id={process.env.NEXT_PUBLIC_GA_ID} />
+        <MicrosoftClarity id={process.env.NEXT_PUBLIC_MC_ID} />
         <LoadFont />
         {languageComponent && <Language />}
       </body>