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

Upgraded nextjs deps and configuration #6388

Merged
merged 5 commits into from
Oct 15, 2024
Merged

Conversation

pnicolli
Copy link
Contributor

@pnicolli pnicolli commented Oct 9, 2024

No description provided.

@pnicolli pnicolli requested a review from sneridagh October 9, 2024 20:34
Copy link

netlify bot commented Oct 9, 2024

Deploy Preview for plone-components canceled.

Name Link
🔨 Latest commit 01e7043
🔍 Latest deploy log https://app.netlify.com/sites/plone-components/deploys/670e7afb4624710008c7be72

@sneridagh
Copy link
Member

@pnicolli I am one of the late "ESM" all the things. How about declare the nextjs app type: module?

@pnicolli
Copy link
Contributor Author

@sneridagh I didn't dig deep into that world yet. What would be the change you are proposing? What I did here is basically to create an empty nextjs project with the suggested tooling in the nextjs docs and copy the config over here 😛

@sneridagh
Copy link
Member

@pnicolli yeah, basically then you don't have to do the .mjs dance anymore. I don't think there are other implications from the app point of view, unless you have to deal with importing .cjs files around.

@pnicolli
Copy link
Contributor Author

@sneridagh I gave it some thought and I think type: module is not a good fit for this app. I would like to keep it as close as possible to out-of-the-box nextjs.

I merged main and fixed a couple eslint warnings, the PR about providers would also benefit from this PR imho.

Copy link
Member

@sneridagh sneridagh left a comment

Choose a reason for hiding this comment

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

LGTM, this can go before the providers refactor

@@ -4,6 +4,7 @@
/node_modules
/.pnp
.pnp.js
.yarn/install-state.gz
Copy link
Member

Choose a reason for hiding this comment

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

?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It's in the latest create-next-app basic template 😅

@pnicolli pnicolli merged commit 9a47ba6 into main Oct 15, 2024
69 of 70 checks passed
@pnicolli pnicolli deleted the pnicolli-upgrade-nextjs branch October 15, 2024 19:56
sneridagh added a commit that referenced this pull request Oct 16, 2024
* main:
  Adjust the path to perform a proper `git diff` between the cached and… (#6410)
  Test whether commenting out the `ignore` command has any affect on the Netlify build.
  support depth in getNavigationQuery (#6400)
  Update caniuse Oct24 (#6408)
  Upgraded nextjs deps and configuration (#6388)
sneridagh added a commit that referenced this pull request Oct 17, 2024
* betterproviders:
  Fix linting
  Changelog
  Latest touches
  Better branding here
  Apply suggestions from code review
  Return whole context instead of only useLocation
  Adjust the path to perform a proper `git diff` between the cached and… (#6410)
  Test whether commenting out the `ignore` command has any affect on the Netlify build.
  Better admonitions format for experimental notices.
  support depth in getNavigationQuery (#6400)
  Update caniuse Oct24 (#6408)
  Upgraded nextjs deps and configuration (#6388)
sneridagh added a commit that referenced this pull request Oct 18, 2024
* main:
  Release generate-volto 9.0.0-alpha.19
  Release @plone/providers 1.0.0-alpha.3
  Release @plone/components 2.0.0-alpha.16
  Release @plone/client 1.0.0-alpha.19
  Release @plone/types 1.0.0-alpha.21
  Some types improvements/fixes (#6412)
  [providers] Improve and group providers (#6069)
  Adjust the path to perform a proper `git diff` between the cached and… (#6410)
  Test whether commenting out the `ignore` command has any affect on the Netlify build.
  support depth in getNavigationQuery (#6400)
  Update caniuse Oct24 (#6408)
  Upgraded nextjs deps and configuration (#6388)
  Update `typescript` and `vitest` everywhere (#6407)
  Remove all traces of Jest 24 in core (#6406)
  Release @plone/components 2.0.0-alpha.15
  [components] Fix Select component logic to support `items` use case (#6405)
  Fix clean make command (#6403)
sneridagh added a commit that referenced this pull request Oct 18, 2024
* main: (37 commits)
  Release generate-volto 9.0.0-alpha.19
  Release @plone/providers 1.0.0-alpha.3
  Release @plone/components 2.0.0-alpha.16
  Release @plone/client 1.0.0-alpha.19
  Release @plone/types 1.0.0-alpha.21
  Some types improvements/fixes (#6412)
  [providers] Improve and group providers (#6069)
  Adjust the path to perform a proper `git diff` between the cached and… (#6410)
  Test whether commenting out the `ignore` command has any affect on the Netlify build.
  support depth in getNavigationQuery (#6400)
  Update caniuse Oct24 (#6408)
  Upgraded nextjs deps and configuration (#6388)
  Update `typescript` and `vitest` everywhere (#6407)
  Remove all traces of Jest 24 in core (#6406)
  Release @plone/components 2.0.0-alpha.15
  [components] Fix Select component logic to support `items` use case (#6405)
  Fix clean make command (#6403)
  Release 18.0.0-alpha.46
  Release @plone/components 2.0.0-alpha.14
  Add missing export for new components (#6391)
  ...
sneridagh added a commit that referenced this pull request Oct 22, 2024
* main: (23 commits)
  Aliases control panel: Improve layout and add CSV upload (#6421)
  Fix site setup access at toolbar by using @actions endpoint. (#6413)
  Remove last CI tests still in 18 (#6418)
  Node 22 as LTS, main and recommended Volto version (#6371)
  Refactor Aliases control panel and support new filters (#6414)
  Remove the Plone app from apps folder (#6415)
  Release generate-volto 9.0.0-alpha.19
  Release @plone/providers 1.0.0-alpha.3
  Release @plone/components 2.0.0-alpha.16
  Release @plone/client 1.0.0-alpha.19
  Release @plone/types 1.0.0-alpha.21
  Some types improvements/fixes (#6412)
  [providers] Improve and group providers (#6069)
  Adjust the path to perform a proper `git diff` between the cached and… (#6410)
  Test whether commenting out the `ignore` command has any affect on the Netlify build.
  support depth in getNavigationQuery (#6400)
  Update caniuse Oct24 (#6408)
  Upgraded nextjs deps and configuration (#6388)
  Update `typescript` and `vitest` everywhere (#6407)
  Remove all traces of Jest 24 in core (#6406)
  ...
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.

2 participants