Skip to content

fix: update nextjs example#351

Open
SgtPooki wants to merge 3 commits into
mainfrom
update/helia-nextjs-example
Open

fix: update nextjs example#351
SgtPooki wants to merge 3 commits into
mainfrom
update/helia-nextjs-example

Conversation

@SgtPooki
Copy link
Copy Markdown
Contributor

@SgtPooki SgtPooki commented Oct 24, 2024

A few fixes here:

  • updates all deps to latest
  • updates the README.md to recommend npm run dev instead of npm start after npm install
  • updates next.config.js to use polyfill recommendation based on Attempted import error: 'RTCSessionDescription' is not exported ipfs/helia#553 (comment)
    • This error cannot be reproduced with latest Helia
  • removes output: 'export' from being used by default because npm run build doesn't build a proper dist folder for use by next start unless output is set to 'standalone' or undefined
  • updates the test script to use next config output of 'export' so tests work properly

@SgtPooki SgtPooki changed the title chore: update nextjs example fix: update nextjs example Oct 25, 2024
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.

1 participant