Fix sentry error with cookiebot #3862 (#3874)

* WIP next-root

* Fix Type #3862

* Fix typo #3862

* Make the layout static to see if it flashes #3862

* Right way #3862

* Fix build #3862

* Duplicate script #3862

Author: padms

web/app/[locale]/(pages)/[...slug]/page.tsx

@@ -4,7 +4,7 @@ import type { Metadata } from 'next'
 import dynamic from 'next/dynamic'
 import { draftMode } from 'next/headers'
 import { notFound } from 'next/navigation'
-import { setRequestLocale } from 'next-intl/server'
+import { locale as rootLocale } from 'next/root-params'
 import { getValidLanguagesLocales } from '@/languageConfig'
 import { decodeSlugs } from '@/lib/helpers/getFullUrl'
 import { Flags } from '@/sanity/helpers/datasetHelpers'
@@ -30,12 +30,12 @@ const NewsPage = dynamic(() => import('@/templates/news/News'))
 const TopicPage = dynamic(() => import('@/templates/topic/TopicPage'))
 const MagazineRoom = dynamic(() => import('@/templates/magazine/Magazineroom'))
 
-export async function generateMetadata({ params }: Props): Promise<Metadata> {
+export async function generateMetadata({
+  params,
+}: PageProps<'/[locale]/[...slug]'>): Promise<Metadata> {
   //array, separated by /. e.g. [news, last slug]
-  const { slug: encodedSlug, locale } = await params
-
-  if (!getValidLanguagesLocales().includes(locale)) notFound()
-
+  const { slug: encodedSlug } = await params
+  const locale = await rootLocale()
   const slug = decodeSlugs(encodedSlug) as string[]
 
   const sanityLang = getNameFromIso(locale)
@@ -80,10 +80,6 @@ export default async function Page({ params, searchParams }: Props) {
   /*   const isInPresentationToolContext =
     (await cookies()).get('preview-fetch-dest')?.value === 'iframe' */
   const { isEnabled: isDraftMode } = await draftMode()
-
-  if (!getValidLanguagesLocales().includes(locale)) notFound()
-
-  setRequestLocale(locale)
   let pageContent = null
   const [siteMenuResult, pageResults] = await Promise.all([
     routeSanityFetch({

web/app/[locale]/(pages)/layout.tsx

@@ -1,25 +1,9 @@
-import { notFound } from 'next/navigation'
-import { hasLocale } from 'next-intl'
-import { setRequestLocale } from 'next-intl/server'
-import { routing } from '@/i18n/routing'
 import { FriendlyCaptchaSdkWrapper } from '../FriendlyCaptchaWrapper'
 
-type Params = Promise<{ locale: string }>
-
-export default async function PagesLayout({
+export default function PagesLayout({
   children,
-  params,
 }: {
   children: React.ReactNode
-  params: Params
 }) {
-  const { locale } = await params
-
-  setRequestLocale(locale)
-
-  if (!hasLocale(routing.locales, locale)) {
-    notFound()
-  }
-
   return <FriendlyCaptchaSdkWrapper>{children}</FriendlyCaptchaSdkWrapper>
 }

web/app/[locale]/(pages)/news/archive/[...slug]/layout.tsx

@@ -1,21 +1,9 @@
 import { notFound } from 'next/navigation'
-import { hasLocale } from 'next-intl'
-import { routing } from '@/i18n/routing'
 import { Flags } from '@/sanity/helpers/datasetHelpers'
-
-type Params = Promise<{ locale: string; slug: string[] }>
-
 export default async function LocaleLayout({
   children,
-  params,
-}: {
-  children: React.ReactNode
-  params: Params
-}) {
-  // Ensure that the incoming `locale` is valid
-  const { locale } = await params
-
-  if (!hasLocale(routing.locales, locale) || !Flags.HAS_ARCHIVED_NEWS) {
+}: LayoutProps<'/[locale]/news/archive/[...slug]'>) {
+  if (!Flags.HAS_ARCHIVED_NEWS) {
     notFound()
   }
 

web/app/[locale]/(pages)/news/page.tsx

@@ -3,7 +3,8 @@ import { algoliasearch } from 'algoliasearch'
 import type { Metadata } from 'next'
 import { unstable_cache } from 'next/cache'
 import dynamic from 'next/dynamic'
-import { setRequestLocale } from 'next-intl/server'
+import { notFound } from 'next/navigation'
+import { locale as rootLocale } from 'next/root-params'
 import { algolia } from '@/lib/config'
 import { Flags } from '@/sanity/helpers/datasetHelpers'
 import { getNameFromIso } from '@/sanity/helpers/localization'
@@ -17,19 +18,8 @@ import NewsRoomTemplate from '@/templates/newsroom/Newsroom'
 
 const TopicPage = dynamic(() => import('@/templates/topic/TopicPage'))
 
-export async function generateStaticParams() {
-  if (Flags.HAS_NEWSROOM) {
-    return [{ locale: 'en-GB' }]
-  }
-  return []
-}
-
-export async function generateMetadata({
-  params,
-}: {
-  params: Promise<{ locale: string }>
-}): Promise<Metadata> {
-  const { locale } = await params
+export async function generateMetadata(): Promise<Metadata> {
+  const locale = await rootLocale()
   const pageSlug = newsSlug[getNameFromIso(locale)]
   if (Flags.HAS_NEWSROOM) {
     const { data: metaData }: { data: any } = await routeSanityFetch({
@@ -47,48 +37,41 @@ export async function generateMetadata({
   return constructSanityMetadata(pageSlug, locale, undefined)
 }
 
-const getInitialResponse = unstable_cache(
-  // this gets revalidated by path
-  async (locale: string) => {
-    const envPrefix = Flags.IS_GLOBAL_PROD ? 'prod' : 'dev'
-    const indexName = `${envPrefix}_NEWS_${locale}`
+export default async function NewsroomPage(_: PageProps<'/[locale]/news'>) {
+  const locale = await rootLocale()
 
-    console.log(
-      new Date(),
-      'Fetching initial response for',
-      indexName,
-      'after revalidation',
-    )
-    const searchClient = algoliasearch(
-      algolia.applicationId,
-      algolia.searchApiKey,
-    )
-    const response = await searchClient.searchSingleIndex({
-      indexName: indexName,
-      searchParams: {
-        hitsPerPage: 50,
-        facetFilters: ['type:news', 'topicTags:-Crude Oil Assays'],
-        facetingAfterDistinct: true,
-        facets: ['countryTags', 'topicTags', 'year'],
-      },
-    })
-    return response
-  },
-  undefined,
-  {
-    tags: [`newsroom_en-GB`],
-  },
-)
-
-export default async function NewsroomPage({
-  params,
-}: {
-  params: Promise<{ slug: string; locale: string }>
-}) {
-  const { locale, slug } = await params
-  // Enable static rendering
-  setRequestLocale(locale)
+  const getInitialResponse = unstable_cache(
+    // this gets revalidated by path
+    async () => {
+      const envPrefix = Flags.IS_GLOBAL_PROD ? 'prod' : 'dev'
+      const indexName = `${envPrefix}_NEWS_${locale}`
 
+      console.log(
+        new Date(),
+        'Fetching initial response for',
+        indexName,
+        'after revalidation',
+      )
+      const searchClient = algoliasearch(
+        algolia.applicationId,
+        algolia.searchApiKey,
+      )
+      const response = await searchClient.searchSingleIndex({
+        indexName: indexName,
+        searchParams: {
+          hitsPerPage: 50,
+          facetFilters: ['type:news', 'topicTags:-Crude Oil Assays'],
+          facetingAfterDistinct: true,
+          facets: ['countryTags', 'topicTags', 'year'],
+        },
+      })
+      return response
+    },
+    undefined,
+    {
+      tags: [`newsroom`, await locale],
+    },
+  )
   const [siteMenuResult, pageResults] = await Promise.all([
     routeSanityFetch({
       query: Flags.HAS_FANCY_MENU ? globalMenuQuery : simpleMenuQuery,
@@ -97,7 +80,7 @@ export default async function NewsroomPage({
       },
     }),
     getPage({
-      slug: slug ?? newsSlug[getNameFromIso(locale)],
+      slug: newsSlug[getNameFromIso(locale)],
       locale,
       tags: ['newsroom'],
     }),
@@ -106,9 +89,10 @@ export default async function NewsroomPage({
   const { headerData, pageData } = pageResults
   const { data: siteMenuData } = siteMenuResult || {}
 
-  const response = Flags.HAS_NEWSROOM
-    ? await getInitialResponse(locale)
-    : undefined
+  const response =
+    Flags.HAS_NEWSROOM && locale in ['en-GB', 'nb-NO']
+      ? await getInitialResponse()
+      : undefined
 
   return (
     <>