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

fix: set default language #2722

Merged
merged 1 commit into from
Jun 6, 2023

Conversation

limerickgds
Copy link
Contributor

@limerickgds limerickgds commented May 17, 2023

Types

  • 🐛 Bug Fixes

Background or solution

🤖 Generated by Copilot at b3f6811

  • Respect user-defined language preference over default preference in getPreferenceLanguageId function (link)

close #2713

Changelog

🤖 Generated by Copilot at b3f6811

Fixed a bug in core-browser that caused the user's language preference to be ignored. Modified getPreferenceLanguageId to compare the scope of the external and default preferences.

@CLAassistant
Copy link

CLAassistant commented May 17, 2023

CLA assistant check
All committers have signed the CLA.

@opensumi opensumi bot added the 🐞 bug Something isn't working label May 17, 2023
@limerickgds limerickgds force-pushed the fix/set-default-language branch from b3f6811 to 71a68ae Compare May 18, 2023 02:09
@erha19
Copy link
Member

erha19 commented May 18, 2023

/next

@limerickgds limerickgds force-pushed the fix/set-default-language branch from 71a68ae to 53b15cc Compare May 18, 2023 02:14
@opensumi
Copy link
Contributor

opensumi bot commented May 18, 2023

🎉 PR Next version 2.24.3-next-1684376157.0 publish successful! You can install this version via npm install package@2.24.3-next-1684376157.0

2.24.3-next-1684376157.0

@Ricbet Ricbet self-requested a review May 22, 2023 02:21
erha19
erha19 previously approved these changes May 24, 2023
@limerickgds
Copy link
Contributor Author

@erha19 这个下个版本会合并吗?

@erha19
Copy link
Member

erha19 commented May 26, 2023

@limerickgds 会的,CI 还有问题,等通过了会合并

@erha19
Copy link
Member

erha19 commented Jun 2, 2023

@limerickgds 这个测试需要修一下
image

@limerickgds limerickgds force-pushed the fix/set-default-language branch from 53b15cc to c67d5f6 Compare June 5, 2023 06:34
@erha19
Copy link
Member

erha19 commented Jun 6, 2023

@limerickgds 还是不对,这里是 packages/core-browser/__tests__/external-preference.test.ts 内的测试会有冲突,需要修改一下

Copy link
Member

@erha19 erha19 left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@erha19 erha19 left a comment

Choose a reason for hiding this comment

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

LGTM

@codecov
Copy link

codecov bot commented Jun 6, 2023

Codecov Report

Patch coverage: 75.00% and project coverage change: -0.05 ⚠️

Comparison is base (b085e3f) 57.72% compared to head (fc55851) 57.68%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2722      +/-   ##
==========================================
- Coverage   57.72%   57.68%   -0.05%     
==========================================
  Files        1327     1333       +6     
  Lines       83563    83846     +283     
  Branches    17381    17432      +51     
==========================================
+ Hits        48238    48366     +128     
- Misses      32104    32249     +145     
- Partials     3221     3231      +10     
Flag Coverage Δ
jsdom 52.70% <75.00%> (-0.03%) ⬇️
node 16.77% <75.00%> (-0.03%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
.../core-browser/src/preferences/early-preferences.ts 89.47% <75.00%> (-1.44%) ⬇️

... and 66 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@erha19 erha19 merged commit 0736342 into opensumi:main Jun 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐞 bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] ClientApp 配置 defaultPreferences['general.language'] 不能立即生效
3 participants