-
-
Notifications
You must be signed in to change notification settings - Fork 999
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix(module: locale): fix custom locale resource #1988
fix(module: locale): fix custom locale resource #1988
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1988 +/- ##
==========================================
+ Coverage 25.51% 27.43% +1.91%
==========================================
Files 488 511 +23
Lines 32111 24509 -7602
Branches 0 233 +233
==========================================
- Hits 8192 6723 -1469
+ Misses 23919 17750 -6169
- Partials 0 36 +36
Continue to review full report at Codecov.
|
docs/i18n.en-US.md
Outdated
@@ -28,11 +28,15 @@ By default, the current language is fetched from the browser. If you need to use | |||
|
|||
``` | |||
|
|||
The default is set to "en-US" when the language you set does not have a built-in language pack. | |||
You can also add it by using the 'LocaleProvider.SetLocale' method. | |||
When the language you set does not have a built-in language pack, we will fallback to the parent `CultureInfo`, then `LocaleProvider.DefaultLanguage`, then "en-US". |
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.
It would be a good idea to add a link to the official documentation for parent CultureInfo
.
* fix(module: locale): fix custom locale resource * update doc * Update LocaleProvider,cs * Update i18n.zh-CN.md * Update LocaleProvider * Update docs
* fix(module: locale): fix custom locale resource * update doc * Update LocaleProvider,cs * Update i18n.zh-CN.md * Update LocaleProvider * Update docs
* fix(module: locale): fix custom locale resource * update doc * Update LocaleProvider,cs * Update i18n.zh-CN.md * Update LocaleProvider * Update docs
* fix(module: locale): fix custom locale resource * update doc * Update LocaleProvider,cs * Update i18n.zh-CN.md * Update LocaleProvider * Update docs
馃 This is a ...
馃敆 Related issue link
Fixes #1980
馃挕 Background and solution
Changes:
LocaleProvider.DefaultLanguage
resource >en-US
馃摑 Changelog
鈽戯笍 Self Check before Merge