ui icon indicating copy to clipboard operation
ui copied to clipboard

Remove `export` statement from `fontSans` variable, it causes type error

Open nguyenyou opened this issue 1 year ago • 1 comments

Screenshot 2024-02-14 at 12 56 41 AM

info  - Need to disable some ESLint rules? Learn more here: https://nextjs.org/docs/basic-features/eslint#disabling-rules
00:54:08.501 | Failed to compile.
00:54:08.502 |  
00:54:08.503 | app/layout.tsx
00:54:08.503 | Type error: Layout "app/layout.tsx" does not match the required types of a Next.js Layout.
00:54:08.503 | "fontSans" is not a valid Layout export field.
00:54:08.503 |  
00:54:08.533 | ELIFECYCLE  Command failed with exit code 1.
00:54:08.557 | Error: Command "pnpm run build" exited with 1
00:54:08.876

<br class="Apple-interchange-newline">

It's weird that no one talks about this, should we update the documentation to avoid this error.

nguyenyou avatar Feb 13 '24 17:02 nguyenyou

@nguyenyou is attempting to deploy a commit to the shadcn-pro Team on Vercel.

A member of the Team first needs to authorize it.

vercel[bot] avatar Feb 13 '24 17:02 vercel[bot]

https://github.com/shadcn-ui/ui/pull/2673 both not merged yet

alissa-tung avatar Feb 17 '24 04:02 alissa-tung

@nguyenyou We fixed it in https://github.com/shadcn-ui/ui/pull/2673. Thanks for your help on this. Appreciate it.

shadcn avatar Mar 03 '24 10:03 shadcn