refactor: Add typeof window check to createConfig #976
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
|
@felixmosh Mind a quick review? The tests make sound assertions, but could be cleaned up in the future in terms of client vs server. However, I think this is fine for now. |
felixmosh
approved these changes
Feb 24, 2021
| @@ -59,7 +61,8 @@ describe('appWithTranslation', () => { | |||
| expect(args[0].i18n.language).toEqual('en') | |||
| expect(args[0].i18n.isInitialized).toEqual(true) | |||
|
|
|||
| expect(fs.existsSync).toHaveBeenCalledTimes(1) | |||
| expect(fs.readdirSync).toHaveBeenCalledTimes(1) | |||
| expect(fs.existsSync).toHaveBeenCalledTimes(0) | |||
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.
No description provided.