Skip to content

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

Merged
merged 13 commits into from
Apr 28, 2022
Merged

feat: Add support for next-i18next #1331

merged 13 commits into from
Apr 28, 2022

Conversation

sarahetter
Copy link

Summary

Adds support for next-i18next out of the box by including the locale files (public/locales/**) and next-i18next.config.js

Test plan

  1. Visit the Deploy Preview (https://next-i18next-test.netlify.app/)

Relevant links (GitHub issues, Notion docs, etc.) or a picture of cute animal

Closes #1241
Screen Shot 2022-04-27 at 3 36 40 PM

Standard checks:

  • Check the Deploy Preview's Demo site for your PR's functionality
  • Add docs when necessary

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

@sarahetter sarahetter requested a review from a team April 27, 2022 19:42
@sarahetter sarahetter self-assigned this Apr 27, 2022
@netlify
Copy link

netlify bot commented Apr 27, 2022

Deploy Preview for netlify-plugin-nextjs-demo ready!

Name Link
🔨 Latest commit 28b796f
🔍 Latest deploy log https://app.netlify.com/sites/netlify-plugin-nextjs-demo/deploys/626ae7fca7f6c10009362cf3
😎 Deploy Preview https://deploy-preview-1331--netlify-plugin-nextjs-demo.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@netlify
Copy link

netlify bot commented Apr 27, 2022

Deploy Preview for netlify-plugin-nextjs-nx-monorepo-demo ready!

Name Link
🔨 Latest commit 28b796f
🔍 Latest deploy log https://app.netlify.com/sites/netlify-plugin-nextjs-nx-monorepo-demo/deploys/626ae7fc2865fb00098979f1
😎 Deploy Preview https://deploy-preview-1331--netlify-plugin-nextjs-nx-monorepo-demo.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@github-actions github-actions bot added the type: feature code contributing to the implementation of a feature and/or user facing functionality label Apr 27, 2022
@netlify
Copy link

netlify bot commented Apr 27, 2022

Deploy Preview for next-plugin-edge-middleware canceled.

Name Link
🔨 Latest commit 28b796f
🔍 Latest deploy log https://app.netlify.com/sites/next-plugin-edge-middleware/deploys/626ae7fc74c681000800e419

@netlify
Copy link

netlify bot commented Apr 27, 2022

Deploy Preview for netlify-plugin-nextjs-static-root-demo ready!

Name Link
🔨 Latest commit 28b796f
🔍 Latest deploy log https://app.netlify.com/sites/netlify-plugin-nextjs-static-root-demo/deploys/626ae7fcc88062000818edca
😎 Deploy Preview https://deploy-preview-1331--netlify-plugin-nextjs-static-root-demo.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@netlify
Copy link

netlify bot commented Apr 27, 2022

Deploy Preview for next-plugin-rsc-demo canceled.

Name Link
🔨 Latest commit 28b796f
🔍 Latest deploy log https://app.netlify.com/sites/next-plugin-rsc-demo/deploys/626ae7fca7f6c10009362cf8

@netlify
Copy link

netlify bot commented Apr 27, 2022

Deploy Preview for netlify-plugin-nextjs-export-demo ready!

Name Link
🔨 Latest commit 28b796f
🔍 Latest deploy log https://app.netlify.com/sites/netlify-plugin-nextjs-export-demo/deploys/626ae7fc2865fb00098979f6
😎 Deploy Preview https://deploy-preview-1331--netlify-plugin-nextjs-export-demo.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@netlify
Copy link

netlify bot commented Apr 27, 2022

Deploy Preview for netlify-plugin-nextjs-next-auth-demo canceled.

Name Link
🔨 Latest commit 28b796f
🔍 Latest deploy log https://app.netlify.com/sites/netlify-plugin-nextjs-next-auth-demo/deploys/626ae7fca803ee0009bfcd30

@@ -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'),
Copy link
Author

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 😅

@cypress
Copy link

cypress bot commented Apr 27, 2022



Test summary

22 0 0 0


Run details

Project netlify-plugin-nextjs-default-demo
Status Passed
Commit d37d041 ℹ️
Started Apr 28, 2022 7:19 PM
Ended Apr 28, 2022 7:20 PM
Duration 01:34 💡
OS Linux Ubuntu - 20.04
Browser Chrome 100

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

@cypress
Copy link

cypress bot commented Apr 27, 2022



Test summary

2 0 0 0


Run details

Project netlify-plugin-nextjs-nx-monorepo-demo
Status Passed
Commit d37d041 ℹ️
Started Apr 28, 2022 7:19 PM
Ended Apr 28, 2022 7:21 PM
Duration 01:13 💡
OS Linux Ubuntu - 20.04
Browser Chrome 100

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

@cypress
Copy link

cypress bot commented Apr 27, 2022



Test summary

7 0 0 0


Run details

Project netlify-plugin-nextjs-static-demo
Status Passed
Commit d37d041 ℹ️
Started Apr 28, 2022 7:19 PM
Ended Apr 28, 2022 7:20 PM
Duration 01:28 💡
OS Linux Ubuntu - 20.04
Browser Chrome 100

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

Copy link

@ericapisani ericapisani left a 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.

@netlify
Copy link

netlify bot commented Apr 28, 2022

Deploy Preview for next-i18next-demo canceled.

Name Link
🔨 Latest commit 28b796f
🔍 Latest deploy log https://app.netlify.com/sites/next-i18next-demo/deploys/626ae7fcc88062000818edcf

@sarahetter
Copy link
Author

Demo site added at https://next-i18next-demo.netlify.app/, and this has been set up in app.netlify.com and is working correctly. ☺️

✅ Deploy Preview for next-i18next-demo ready!

Name Link
🔨 Latest commit 4f34543
🔍 Latest deploy log https://app.netlify.com/sites/next-i18next-demo/deploys/626aa9ce1a20ad0008c5b8aa
😎 Deploy Preview https://deploy-preview-1331--next-i18next-demo.netlify.app
📱 Preview on mobile
Toggle QR Code...
To edit notification comments on pull requests, go to your Netlify site settings.

@sarahetter sarahetter requested a review from ascorbic April 28, 2022 15:03
Copy link

@nickytonline nickytonline left a 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!

Copy link

@ericapisani ericapisani left a comment

Choose a reason for hiding this comment

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

LGTM! 🚀

@kodiakhq kodiakhq bot merged commit 5780e6b into main Apr 28, 2022
@kodiakhq kodiakhq bot deleted the feat/next-i18next branch April 28, 2022 19:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge type: feature code contributing to the implementation of a feature and/or user facing functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support next-i18next out of the box
4 participants