-
Notifications
You must be signed in to change notification settings - Fork 86
feat: Add support for next-i18next #1331
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
Conversation
✅ Deploy Preview for netlify-plugin-nextjs-demo ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
✅ Deploy Preview for netlify-plugin-nextjs-nx-monorepo-demo ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
✅ Deploy Preview for next-plugin-edge-middleware canceled.
|
✅ Deploy Preview for netlify-plugin-nextjs-static-root-demo ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
✅ Deploy Preview for next-plugin-rsc-demo canceled.
|
✅ Deploy Preview for netlify-plugin-nextjs-export-demo ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
✅ Deploy Preview for netlify-plugin-nextjs-next-auth-demo canceled.
|
@@ -1,9 +1,9 @@ | |||
jest.mock('../plugin/src/helpers/utils', () => { | |||
return { | |||
...(jest.requireActual('../plugin/src/helpers/utils')), | |||
isNextAuthInstalled: jest.fn() | |||
...jest.requireActual('../plugin/src/helpers/utils'), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Extra noise in this file due to my autosave lint plugin. Good news is this file now follows our eslint and prettier rules 😅
Test summaryRun details
View run in Cypress Dashboard ➡️ This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard |
Test summaryRun details
View run in Cypress Dashboard ➡️ This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard |
Test summaryRun details
View run in Cypress Dashboard ➡️ This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is overall looking great!
I had one question, and the only blocker to me approving (unless there's a strong reason not to) is we'll want to have a demo of this site deployed automatically as part of our continuous testing similar to what we have for the other demo sites.
✅ Deploy Preview for next-i18next-demo canceled.
|
Demo site added at https://next-i18next-demo.netlify.app/, and this has been set up in app.netlify.com and is working correctly.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[dust] Since this PR is for adding the next-i18next demo site, consider adding the non-related files to the next-i18next demo in another PR.
Aside from that looks good!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! 🚀
Summary
Adds support for next-i18next out of the box by including the locale files (
public/locales/**
) andnext-i18next.config.js
Test plan
Relevant links (GitHub issues, Notion docs, etc.) or a picture of cute animal
Closes #1241

Standard checks:
🧪 Once merged, make sure to update the version if needed and that it was published correctly.