Skip to content

Add Next.js demo app#35

Merged
sjoerdbeentjes merged 9 commits into
mainfrom
feat/nextjs-app
Jun 23, 2026
Merged

Add Next.js demo app#35
sjoerdbeentjes merged 9 commits into
mainfrom
feat/nextjs-app

Conversation

@sjoerdbeentjes

@sjoerdbeentjes sjoerdbeentjes commented Jun 22, 2026

Copy link
Copy Markdown
Collaborator

Adds a minimal Next.js (App Router) demo app under apps/ that consumes @surfnet/react as a real workspace dependency, smoke-testing that the package imports and renders outside Storybook.

Changes

  • New @surfnet/react-app at apps/react-app/: renders the Button and imports the package's compiled @surfnet/react/styles.css.
  • apps/* added to the pnpm workspace; Turbo wires @surfnet/react-app#build after @surfnet/react#build via ^build.
  • transpilePackages: ['@surfnet/react'] plus outputFileTracingRoot pinned to silence the parent-lockfile warning.
  • sharp: true added to the workspace allowBuilds policy (pulled in by Next).
  • README and AGENTS.md document the demo app.

Verification

  • pnpm --filter @surfnet/react-app build passes (static prerender).
  • Prod server renders the button with its full @surfnet/react variant classes.

Note: requires Node 22 (nvm use); Node 24 breaks corepack/pnpm in this repo.

@sjoerdbeentjes sjoerdbeentjes requested review from loofpc and oharsta June 22, 2026 09:01
@sjoerdbeentjes sjoerdbeentjes changed the title Add @surfweb/react-app demo Next.js app Add Next.js demo app Jun 22, 2026
@sjoerdbeentjes sjoerdbeentjes marked this pull request as draft June 22, 2026 09:04
@sjoerdbeentjes sjoerdbeentjes force-pushed the feat/nextjs-app branch 2 times, most recently from 715020a to 7ac09bc Compare June 22, 2026 09:29
@sjoerdbeentjes sjoerdbeentjes marked this pull request as ready for review June 22, 2026 09:51
@sjoerdbeentjes sjoerdbeentjes marked this pull request as draft June 22, 2026 09:53
@sjoerdbeentjes sjoerdbeentjes marked this pull request as ready for review June 22, 2026 10:53
Comment thread apps/react-app/css.d.ts Outdated
@sjoerdbeentjes sjoerdbeentjes merged commit ca6148e into main Jun 23, 2026
1 check 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