search

Joelius300 / ChartJSBlazor

This library is a modification of the awesome ChartJs.Blazor library by mariusmuntean. It's supposed to add more functionality to the LineChart and generally make the library more complete.
Other
40 stars 6 forks source link

Update to .NET Core 3.0 Preview 8 #57

Closed SeppPenner closed 5 years ago

SeppPenner commented 5 years ago

Check https://devblogs.microsoft.com/aspnet/asp-net-core-and-blazor-updates-in-net-core-3-0-preview-8/ for changes.

Joelius300 commented 5 years ago

Let's do this after https://github.com/Joelius300/ChartJSBlazor/pull/50 and before https://github.com/Joelius300/ChartJSBlazor/issues/28 as mentioned here. I'll do it after merging https://github.com/Joelius300/ChartJSBlazor/pull/50.

After that we really need to create a release to keep up with the changelog and also bring out a new nuget-version. We really have to document all the breaking changes and update the wiki (including samples and readme) before starting to work on other charts or new features!

SeppPenner commented 5 years ago

Most important thing here: https://devblogs.microsoft.com/aspnet/asp-net-core-and-blazor-updates-in-net-core-3-0-preview-8/#comments-702:

In the migration steps, you also need to change any package references of Microsoft.AspNetCore.Components.Browser to Microsoft.AspNetCore.Components.Web https://www.nuget.org/packages/Microsoft.AspNetCore.Components.Web/

This killed my project today...

https://www.nuget.org/packages/Microsoft.AspNetCore.Components.Browser/ seems to be not switched to preview8 (Maybe this will never be updated because everything is in *.Web now?): 3.0.0-preview7.19365.7

SeppPenner commented 5 years ago

It's fixed once https://github.com/Joelius300/ChartJSBlazor/pull/60 is merged and @Joelius300 verified that this can be run properly.