Change environment URL, conditionally add speed insights

codetalkio · Jan 10, 2024 · e1a8a50 · e1a8a50
1 parent 9c6f04c
commit e1a8a50
Show file tree

Hide file tree

Showing 4 changed files with 11 additions and 24 deletions.
diff --git a/.github/workflows/vercel-deployment.yml b/.github/workflows/vercel-deployment.yml
@@ -18,5 +18,7 @@ jobs:
       - name: End-to-end tests
         working-directory: ./deployment
         env:
-          UI_APP_E2E_URL: ${{ github.event.deployment_status.url }}
-        run: just e2e deployment ui-app
+          UI_APP_E2E_URL: ${{ github.event.deployment_status.environment_url }}
+        run: |
+          echo "Running tests against '${UI_APP_E2E_URL}'."
+          just e2e deployment ui-app
diff --git a/ui-app/bun.lockb b/ui-app/bun.lockb
diff --git a/ui-app/package.json b/ui-app/package.json
@@ -9,6 +9,7 @@
     "lint": "next lint"
   },
   "dependencies": {
+    "@vercel/speed-insights": "^1.0.2",
     "next": "14.0.4",
     "next-intl": "3.4.1",
     "react": "18.2.0",

diff --git a/ui-app/src/app/[locale]/layout.tsx b/ui-app/src/app/[locale]/layout.tsx
@@ -1,10 +1,9 @@
 import '../globals.css';
-// import type { Metadata } from 'next';
 import { Inter } from 'next/font/google';
 import { unstable_setRequestLocale, getTranslations } from 'next-intl/server';
-// import { notFound } from 'next/navigation';
 import { ReactNode } from 'react';
 import { promises as fs } from 'fs';
+import { SpeedInsights } from '@vercel/speed-insights/next';
 
 const inter = Inter({ subsets: ['latin'] });
 
@@ -18,18 +17,6 @@ export async function generateStaticParams() {
   return localeFiles.map((f) => ({ locale: f.replace(/\.json$/, '') }));
 }
 
-/**
- * Load the contents of a given locale's messages file.
- */
-// async function messagesContent(locale: string) {
-//   try {
-//     return (await import(`../../../messages/${locale}.json`)).default;
-//   } catch (error) {
-//     console.error('[messagesContent] Something went wrong importing the message content', error);
-//     notFound();
-//   }
-// }
-
 type Params = {
   params: Awaited<ReturnType<typeof generateStaticParams>>[0];
 };
@@ -54,16 +41,13 @@ export default async function Layout({ children, params: { locale } }: Props) {
   // Required workaround for static rendering with next-intl.
   // https://next-intl-docs.vercel.app/docs/getting-started/app-router#static-rendering
   unstable_setRequestLocale(locale);
+
   return (
     <html lang={locale}>
-      <body className={inter.className}>{children}</body>
+      <body className={inter.className}>
+        {children}
+        {process.env.IS_VERCEL_BUILD ? <SpeedInsights /> : null}
+      </body>
     </html>
   );
 }
-
-{
-  /* import { NextIntlClientProvider } from 'next-intl';
-  <NextIntlClientProvider locale={locale} messages={messages}>
-  {children}
-</NextIntlClientProvider> */
-}