refactor: Use serialize-javascript #954
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This was referenced Feb 21, 2021
Closed
|
Hey! I don't love "hammering" the Function call, but hey, if it's safe and it works, I'm all for it. My only question is: how do we know for sure that the config is a function and not an object, for example? |
|
@filipesmedeiros Not sure I catch your meaning – the config is an object. It also happens to contain functions, etc. |
|
Doesn't EDIT: Nevermind, I read the code wrong! |
isaachinman
added a commit
that referenced
this pull request
Feb 24, 2021
* Core changes to support NextJs v10 * Add default config file reading functionality * Reimplement hoistNonReactStatics * Update dependencies * Strip out tests and unnecessary dependencies * Fix linting setup * Upgrade core dependencies * Migrate config options to mirror NextJs * v8.0.0-beta.0 * Remove deprecated config options and types * Update example lockfile * Remove need for locales and defaultLocale entirely * Update example to use hooks * Add documentation * v8.0.0-beta.1 * Add NodeJs v14 to CircleCI * Create serverSideTranslations.d.ts * v8.0.0-beta.2 * fix: TS issues for v8-beta (#922) * fix: export appWithTranslation type also changed AppWithTranslation type: param is a React Component, not a React Component instance * chore: remove unused NextI18Next exports * fix: run-example prod script by adding the build step before it * v8.0.0-beta.3 * fix: Remove server side code from client bundle (#926) * Remove static dir entirely * v8.0.0-beta.4 * feat: Add an ESM entry point to allow better bundle optimizations (#937) * Add an ESM entry point to allow better tree-shaking & module concatenation * Include ie11 * fix: added Trans to TS definition v8-beta (#951) * added Trans to types * fix: removed unused typess * refactor: Use serialize-javascript (#954) * refactor: Use NextJs i18n config API (#955) * test: Add tests for config and consoleMessage (#956) * test: Add coverage for createClient (#957) * refactor: Update eslint config (#958) * refactor: Update eslint config * Fix lint * test: Add e2e coverage via Cypress (#959) * feat: Disable FS backend when user provides one (#930) * feat: Disable FS backend when user provides one * fixup! feat: Disable FS backend when user provides one * fixup! feat: Disable FS backend when user provides one * fixup! feat: Disable FS backend when user provides one * test: Add test coverage to appWithTranslation and serverSideTranslations (#960) * docs: Update README (#961) * v8.0.0-beta.5 * docs: Update README (#962) * v8.0.0-beta.6 * chore: Update example lockfile (#964) * chore: Update example lockfile * Update package.json * fix: Deep merge options.backend, closes #848 (#849) * fix: Make create-config defaultFile use localeStructure (#966) use @cescoallegrini modifications from #934 * Update README.md Co-authored-by: Léonard Drouillas <[email protected]> * chore: Lint examples dir (#969) * fix: Add server side loading of fallbackLng (#970) * refactor: Remove i18next-http-backend (#971) * refactor: Remove i18next-http-backend * Update config.yml * Update basic.e2e.ts * refactor: Add react-i18next useSuspense (#973) * docs: Update README (#975) * refactor: Add typeof window check to createConfig (#976) * refactor: Remove serialize-javascript and add configOverride to appWithTranslation (#972) * refactor: Remove serialize-javascript and add configOverride to appWithTranslation * Add serializeConfig option * Add docs * Update README.md * docs: Update README (#977) * Update peerDependencies and version Co-authored-by: Francesco Moro <[email protected]> Co-authored-by: Felix Mosheev <[email protected]> Co-authored-by: Tobias Koller <[email protected]> Co-authored-by: Filipe Medeiros <[email protected]> Co-authored-by: Léonard Drouillas <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uses serialize-javascript to stringify a user's i18next config, which may contain primitives (like functions) that NextJs will refuse to serialise out of the box.
The config gets serialised in
serverSideTranslations, and then reparsed inappWithTranslationviaFunction().This allows us to maintain a
next-i18next.config.jsfile, without requiring the user to import it themselves, or pass it as arguments into any ofnext-i18next's functions.Fixes #931.