MudBlazor icon indicating copy to clipboard operation
MudBlazor copied to clipboard

TimeSeries: Rename TimeSeriesDiplayType to fix typo

Open danielchalmers opened this issue 1 year ago • 1 comments
trafficstars

Description

How Has This Been Tested?

Type of Changes

  • [ ] Bug fix (non-breaking change which fixes an issue)
  • [ ] New feature (non-breaking change which adds functionality)
  • [X] Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • [ ] Documentation (fix or improvement to the website or code docs)

Checklist

  • [X] The PR is submitted to the correct branch (dev).
  • [X] My code follows the code style of this project.
  • [ ] I've added relevant tests.

danielchalmers avatar Aug 07 '24 15:08 danielchalmers

Codecov Report

Attention: Patch coverage is 75.00000% with 1 line in your changes missing coverage. Please review.

Project coverage is 90.80%. Comparing base (28bc599) to head (0de4eac). Report is 564 commits behind head on dev.

Files with missing lines Patch % Lines
...Blazor/Components/Chart/Charts/TimeSeries.razor.cs 66.66% 0 Missing and 1 partial :warning:
Additional details and impacted files
@@            Coverage Diff             @@
##              dev    #9581      +/-   ##
==========================================
+ Coverage   89.82%   90.80%   +0.97%     
==========================================
  Files         412      411       -1     
  Lines       11878    12844     +966     
  Branches     2364     2483     +119     
==========================================
+ Hits        10670    11663     +993     
+ Misses        681      618      -63     
- Partials      527      563      +36     

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov[bot] avatar Aug 07 '24 16:08 codecov[bot]

Ah wasnt fast enough to close this, as it was already done in this PR: https://github.com/MudBlazor/MudBlazor/pull/9919

ScarletKuro avatar Oct 13 '24 13:10 ScarletKuro

oh, in any case I added it to the v8 migration guide: https://github.com/MudBlazor/MudBlazor/issues/9953

henon avatar Oct 13 '24 13:10 henon