Merge pull request #54 from jaegermoritz/feat/version-and-release-management

fix(web-app): 🐛 fix bugs that break the build

Author: jaegermoritz

.github/workflows/conventional_commits.yml

@@ -1,4 +1,4 @@
-name: "🚨 Lint PR 🚨"
+name: "🚨 Lint PR"
 
 on:
   pull_request_target:
@@ -30,18 +30,18 @@ jobs:
           header: pr-title-lint-error
           message: |
             Hey there and thank you for opening this pull request! 👋🏼
-            
+
             We require pull request titles to follow the [Conventional Commits specification](https://www.conventionalcommits.org/en/v1.0.0/) and it looks like your proposed title needs to be adjusted.
 
             Details:
-            
+
             ```
             ${{ steps.lint_pr_title.outputs.error_message }}
             ```
 
       # Delete a previous comment when the issue has been resolved
       - if: ${{ steps.lint_pr_title.outputs.error_message == null }}
         uses: marocchino/sticky-pull-request-comment@v2
-        with:   
+        with:
           header: pr-title-lint-error
           delete: true

.github/workflows/release.yml

@@ -1,4 +1,4 @@
-name: Release
+name: 🔖 Release
 
 on:
   push:

.vscode/settings.json

@@ -8,6 +8,7 @@
     "graphql-server",
     "deps",
     "database",
-    "release"
+    "release",
+    "web-app"
   ]
 }

apps/docs/next-env.d.ts

@@ -2,4 +2,4 @@
 /// <reference types="next/image-types/global" />
 
 // NOTE: This file should not be edited
-// see https://nextjs.org/docs/basic-features/typescript for more information.
+// see https://nextjs.org/docs/app/building-your-application/configuring/typescript for more information.

apps/web/app/[locale]/layout.tsx

@@ -1,22 +1,23 @@
 import { NextIntlClientProvider } from 'next-intl'
 import { getMessages } from 'next-intl/server'
 import { ApolloWrapper } from '@/apollo-client/ApolloWrapper'
-import { Metadata } from 'next'
+import type { Metadata, ResolvingMetadata } from 'next'
 import { Header } from '@/components/header'
 import { Footer } from '@/components/footer'
 
-export const metaData: Metadata = {
-  title: 'Homepage',
-  description: 'This is the homepage',
-}
-
-export default async function LocaleLayout({
-  children,
-  params: { locale },
-}: {
+type Props = {
   children: React.ReactNode
   params: { locale: string }
-}) {
+}
+
+export async function generateMetadata({ params }: Props, parent: ResolvingMetadata): Promise<Metadata> {
+  return {
+    title: 'Homepage',
+    description: 'This is the homepage',
+  }
+}
+
+export default async function LocaleLayout({ children, params: { locale } }: Props) {
   // Providing all messages to the client
   // side is the easiest way to get started
   const messages = await getMessages()

apps/web/app/api/auth/[...nextauth]/route.ts

@@ -1,2 +1,2 @@
-import { handlers } from '@/auth' // Referring to the auth.ts we just created
+import { handlers } from '../../../../auth'
 export const { GET, POST } = handlers

apps/web/app/api/protected/route.ts

@@ -6,8 +6,12 @@
  *
  * * For more information on how next confiures Jest review this file:
  * {@link https://github.com/vercel/next.js/blob/v14.2.3/packages/next/src/build/jest/jest.ts}
+ *
+ * As the official documentation is not providing a working example for next.js
+ * I reworked the implementation following this discussion:
+ * {@link https://github.com/vercel/next.js/discussions/73894}
  */
-import type { Config } from 'jest'
+import { Config } from '@jest/types'
 import nextJest from 'next/jest.js'
 import { compilerOptions } from './tsconfig.json'
 
@@ -17,7 +21,7 @@ const createJestConfig = nextJest({
 })
 
 // Add any custom config to be passed to Jest
-const config: Config = {
+const config: Config.InitialOptions = {
   collectCoverage: true,
   coverageProvider: 'v8',
 

apps/web/jest.config.ts

@@ -1 +1,4 @@
 import '@testing-library/jest-dom'
+import 'whatwg-fetch'
+import { TextDecoder, TextEncoder } from 'util'
+Object.assign(global, { TextDecoder, TextEncoder })

apps/web/jest.setup.ts

@@ -2,4 +2,4 @@
 /// <reference types="next/image-types/global" />
 
 // NOTE: This file should not be edited
-// see https://nextjs.org/docs/basic-features/typescript for more information.
+// see https://nextjs.org/docs/app/building-your-application/configuring/typescript for more information.

apps/web/next-env.d.ts

@@ -41,12 +41,14 @@
     "@commitlint/config-conventional": "^19.2.2",
     "@graphql-codegen/cli": "^5.0.2",
     "@graphql-codegen/named-operations-object": "^3.0.0",
+    "@jest/types": "^29.6.3",
     "@next/eslint-plugin-next": "^14.1.1",
     "@repo/eslint-config": "workspace:*",
     "@repo/typescript-config": "workspace:*",
     "@testing-library/jest-dom": "^6.4.5",
     "@testing-library/react": "^15.0.7",
     "@types/eslint": "^8.56.5",
+    "@types/jest": "^29.5.14",
     "@types/node": "^20.11.24",
     "@types/react": "^18.2.61",
     "@types/react-dom": "^18.2.19",
@@ -60,7 +62,8 @@
     "prettier": "^3.2.5",
     "ts-jest": "^29.1.2",
     "ts-node": "^10.9.2",
-    "typescript": "^5.4.5"
+    "typescript": "^5.4.5",
+    "whatwg-fetch": "^3.6.20"
   },
   "engines": {
     "node": ">=20"

apps/web/package.json

@@ -11,7 +11,8 @@
     },
     "resolveJsonModule": true,
     "skipLibCheck": true,
-    "strict": true
+    "strict": true,
+    "typeRoots": ["./node_modules/@types"]
   },
   "include": [
     "**/*.config.js",