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

feat(module: statistic): add CultureInfo to support localization number format #3299

Merged
merged 4 commits into from
Aug 19, 2023

Conversation

ElderJames
Copy link
Member

🤔 This is a ...

  • New feature
  • Bug fix
  • Site / documentation update
  • Demo update
  • Component style update
  • Bundle size optimization
  • Performance optimization
  • Refactoring
  • Code style optimization
  • Test Case
  • Branch merge
  • Other (about what?)

🔗 Related issue link

💡 Background and solution

📝 Changelog

Language Changelog
🇺🇸 English add CultureInfo to support localization number format
🇨🇳 Chinese 增加 CultureInfo 属性来支持不同地区的数字格式

☑️ Self Check before Merge

⚠️ Please check all items below before review. ⚠️

  • Doc is updated/provided or not needed
  • Demo is updated/provided or not needed
  • Changelog is provided or not needed

@github-actions
Copy link

github-actions bot commented Jun 12, 2023

@codecov
Copy link

codecov bot commented Jun 12, 2023

Codecov Report

Patch coverage: 32.12% and project coverage change: -1.42% ⚠️

Comparison is base (7af4adc) 46.70% compared to head (5736132) 45.29%.
Report is 44 commits behind head on feature.

Additional details and impacted files
@@             Coverage Diff             @@
##           feature    #3299      +/-   ##
===========================================
- Coverage    46.70%   45.29%   -1.42%     
===========================================
  Files          557      558       +1     
  Lines        26731    26981     +250     
  Branches       266      267       +1     
===========================================
- Hits         12486    12220     -266     
- Misses       14205    14721     +516     
  Partials        40       40              
Files Changed Coverage Δ
components/card/Card.razor 76.19% <ø> (ø)
components/collapse/Panel.razor 0.00% <0.00%> (ø)
components/collapse/Panel.razor.cs 0.00% <0.00%> (ø)
components/core/HashCodes/HashCodeExtensions.cs 0.00% <0.00%> (ø)
components/core/HashCodes/HashCodeProvider.cs 0.00% <0.00%> (ø)
components/core/HashCodes/ParameterDescriptor.cs 0.00% <ø> (ø)
components/core/JsInterop/DomEventListener.cs 0.00% <0.00%> (ø)
...s/core/JsInterop/modules/dom/manipulationHelper.ts 13.87% <0.00%> (-0.17%) ⬇️
components/date-picker/DatePicker.razor 77.77% <ø> (ø)
components/date-picker/RangePicker.razor 54.54% <ø> (+6.06%) ⬆️
... and 39 more

... and 19 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@ElderJames ElderJames changed the base branch from master to feature July 9, 2023 09:04
@ElderJames ElderJames merged commit 0a6d75a into feature Aug 19, 2023
4 of 6 checks passed
@ElderJames ElderJames deleted the feat/statistic-cultureinfo branch August 19, 2023 14:17
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.

None yet

1 participant