mlim-usfca / PersonalKnowledge

https://personal-knowledge.vercel.app
0 stars 0 forks source link

Fix `build` for Web #55

Closed inhwaS closed 4 months ago

inhwaS commented 5 months ago

We are not able to run yarn build command with those error.

inhwa.son@inhwas-air web % yarn build
yarn run v1.22.19
warning ../package.json: No license field
warning ../../package.json: No license field
$ next build
  ▲ Next.js 14.2.2
  - Environments: .env.local

   Creating an optimized production build ...
 ✓ Compiled successfully

./src/app/auth/callback/page.tsx
38:6  Warning: React Hook useEffect has a missing dependency: 'isAuthenticated'. Either include it or remove the dependency array.  react-hooks/exhaustive-deps

./src/app/chats/chat.tsx
77:8  Warning: React Hook useEffect has a missing dependency: 'chat.messages'. Either include it or remove the dependency array.  react-hooks/exhaustive-deps
77:9  Warning: React Hook useEffect has a complex expression in the dependency array. Extract it to a separate variable so it can be statically checked.  react-hooks/exhaustive-deps
83:8  Warning: React Hook useEffect has a missing dependency: 'dispatch'. Either include it or remove the dependency array.  react-hooks/exhaustive-deps
171:33  Warning: Using `<img>` could result in slower LCP and higher bandwidth. Consider using `<Image />` from `next/image` to automatically optimize images. This may incur additional usage or cost from your provider. See: https://nextjs.org/docs/messages/no-img-element  @next/next/no-img-element

./src/app/dashboard/page.tsx
26:6  Warning: React Hook useLayoutEffect has missing dependencies: 'isAuthenticated' and 'router'. Either include them or remove the dependency array.  react-hooks/exhaustive-deps

./src/app/page.tsx
20:6  Warning: React Hook useLayoutEffect has missing dependencies: 'isAuthenticated' and 'router'. Either include them or remove the dependency array.  react-hooks/exhaustive-deps

./src/components/savedcontent.tsx
35:8  Warning: React Hook useEffect has missing dependencies: 'dispatch', 'router', and 'user'. Either include them or remove the dependency array.  react-hooks/exhaustive-deps

./src/components/user.tsx
20:21  Warning: Using `<img>` could result in slower LCP and higher bandwidth. Consider using `<Image />` from `next/image` to automatically optimize images. This may incur additional usage or cost from your provider. See: https://nextjs.org/docs/messages/no-img-element  @next/next/no-img-element

info  - Need to disable some ESLint rules? Learn more here: https://nextjs.org/docs/basic-features/eslint#disabling-rules
   Linting and checking validity of types  ..Failed to compile.

./src/app/api.ts:2:31
Type error: Module '"./interfaces"' has no exported member 'SavedCategory'.

  1 | import axios, { AxiosResponse } from 'axios';
> 2 | import { UserData, SavedLink, SavedCategory, Tags } from './interfaces';
    |                               ^
  3 | import { mockUser, mockLinks, mockTags } from './mockData';
  4 | import { supabase } from '../components/supabase';
  5 |
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
TanyaaCJain commented 4 months ago

Closing as addressed in #59.