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

Add new property extensionsRgx to config #589

Merged
merged 2 commits into from
Apr 22, 2021

Conversation

berndartmueller
Copy link
Contributor

To be able to use custom Next.js page extensions (vercel/next.js#8454 (comment)), a new property extensionsRgx is added to the i18n configuration.

How to use

// i18n.js

module.exports = {
  extensionsRgx: /\.(page|api)\.(tsx|ts|js|mjs|jsx)$/,
  ...
}

To be able to use custom Next.js page extensions (vercel/next.js#8454 (comment)), a new property `extensionsRgx` is added to the i18n configuration.

# How to use

// i18n.js

```
module.exports = {
  extensionsRgx: /\.(page|api)\.(tsx|ts|js|mjs|jsx)$/,
  ...
}
```
Copy link
Owner

@aralroca aralroca left a comment

Choose a reason for hiding this comment

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

@berndartmueller sounds good! And welcome as a first time contributor!!

Please, would you add the property in the configuration table of README.md docs? Thank you very much!!

@aralroca aralroca added this to the 1.0.6 milestone Apr 22, 2021
@berndartmueller
Copy link
Contributor Author

Please, would you add the property in the configuration table of README.md docs? Thank you very much!!

Sure! Just pushed the change.

@aralroca Thanks for your super quick response!

@aralroca aralroca merged commit d42dfdd into aralroca:canary Apr 22, 2021
@aralroca aralroca modified the milestones: 1.0.6, 1.0.7 Apr 22, 2021
@aralroca
Copy link
Owner

@berndartmueller I pre-released 1.0.7-canary.1 with this change if you want to start using it. It will be released on the next release.

@aralroca
Copy link
Owner

@allcontributors please add @berndartmueller for code

@allcontributors
Copy link
Contributor

@aralroca

I've put up a pull request to add @berndartmueller! 🎉

aralroca added a commit that referenced this pull request May 21, 2021
* replace quotes only for the special $' pattern specific to .replace() (#529)

* replace quotes only for the special $' pattern specific to .replace() instead of replacing quoutes everywhere

* use .replace() callback to avoid parsing special string patterns

* write tests to verify that templateWithHoc and templateWithLoader correctly replaces special string cases. Update snapshots

* Update package version

* fix(transCore): when no suffix don't match spaces (#534)

* Update package version

* _one works (#541)

* Update package version

* Update README.md (#552)

* Update package version

* Update dependencies (#554)

* Update dependencies

* Update example deps

* Update Trans text after change lang (#566)

* Ignore api.(ts|js...) file (#567)

* Add useMemo to useTranslation (#574)

* Update deps (#582)

* Update version of package.json

* Adding tests (#585)

* Add new property `extensionsRgx` to config (#589)

* Add new property `extensionsRgx` to config

To be able to use custom Next.js page extensions (vercel/next.js#8454 (comment)), a new property `extensionsRgx` is added to the i18n configuration.

# How to use

// i18n.js

```
module.exports = {
  extensionsRgx: /\.(page|api)\.(tsx|ts|js|mjs|jsx)$/,
  ...
}
```

* docs: add `extensionsRgx` configuration property

* Update package.json

* Listen for `namespaces` changes and load necessary namespaces (#592)

* Update package.json version

* Revert "Add useMemo to useTranslation" (#605)

This reverts commit 8abc458.

# Conflicts:
#	package.json

* Remove console.warn because is already solved on Next.js 10.2.1-canary.4 (#609)

Already fixed in [Next.js canary 10.2.1-canary.4](https://github.com/vercel/next.js/releases/tag/v10.2.1-canary.4)

Co-authored-by: AndrewB <[email protected]>
Co-authored-by: slevy85 <[email protected]>
Co-authored-by: Justin <[email protected]>
Co-authored-by: Bernd Artmüller <[email protected]>
Co-authored-by: Rihards Ščeredins <[email protected]>
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