-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
fix(locale): 修复 locale 空配置时 antd 缺失部分内容的情况 #10762
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 1 Ignored Deployment
|
Size Change: -8 B (0%) Total Size: 9.73 MB
ℹ️ View Unchanged
|
Codecov ReportPatch and project coverage have no change.
Additional details and impacted files@@ Coverage Diff @@
## master #10762 +/- ##
=======================================
Coverage 29.11% 29.11%
=======================================
Files 462 462
Lines 13871 13871
Branches 3253 3253
=======================================
Hits 4038 4038
Misses 9143 9143
Partials 690 690
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report in Codecov by Sentry. |
@@ -146,7 +146,7 @@ export default (api: IApi) => { | |||
|
|||
let DefaultAntdLocales: string[] = []; | |||
// set antd default locale | |||
if (!antdLocales.length && api.config.locale?.antd) { | |||
if (!antdLocales.length && antd) { |
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.
这个 antd 在哪里声明的?
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.
上面几行声明的,antd 会综合考虑 config 和上面的 defaultConfig,其他地方也是用 antd 判断的,唯独这里感觉是遗漏掉了
当仅配置
locale: {}
时,会得到: