Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: update nextjs example #351

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Conversation

SgtPooki
Copy link
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