Skip to content

feat: upgrade to Next.js 16 and add shadcn/ui components - #1

Merged
milosh86 merged 5 commits into
milosh86:mainfrom
ZeviLiao:feats/upgrade-nextjs16
Mar 16, 2026
Merged

milosh86 merged 5 commits into
milosh86:mainfrom
ZeviLiao:feats/upgrade-nextjs16

Conversation

@ZeviLiao

Copy link
Copy Markdown
Contributor

Changes

  • Upgraded Next.js to v16.1
  • Added shadcn/ui with Button component
  • Configured Tailwind CSS v3.4 (v4 not compatible with Turbopack)
  • Added component tests using Vitest and React Testing Library
  • Updated README with tech stack and known issues

Tests

  • ✅ Button component tests (6 passing)
  • ✅ Home page test (1 passing)
  • ✅ All tests passing (7/7)

Notes

Tailwind CSS v4 has compatibility issues with Next.js 16 + Turbopack, so we're using v3.4 for now.

@milosh86 milosh86 left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @ZeviLiao, looks good, left a couple of minor comments.

Comment thread src/app/globals.css
Comment thread package.json Outdated
- Restore original CSS variable order and structure in globals.css
- Remove unused dev:webpack script from package.json

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

@milosh86 milosh86 left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @ZeviLiao , it's looking good 👍

@milosh86

Copy link
Copy Markdown
Owner

@ZeviLiao could you please update storybook and its related dependencies to the latest version? ... CI fails right now

@chromatic-com/storybook@4.x requires storybook@^9||^10, which conflicted
with the pinned storybook@8.x, causing npm ci to fail in CI. Upgraded all
storybook packages to v10 and removed addons now bundled in core.
@milosh86
milosh86 merged commit 86160c1 into milosh86:main Mar 16, 2026
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants