nexus-main / nexus

Extensible user interface for your time-series data lake with REST API support.
MIT License
4 stars 5 forks source link

fixing (Missing Nullable Statement in MudBlazor results in Build warning #105) #107

Closed joe97z closed 3 months ago

joe97z commented 3 months ago

adding if statement for the value if is not "Null"

Apollo3zehn commented 3 months ago

Hi, thanks for your PR. The problem with your proposed solution is that now null values are not being set anymore although they are valid values in the general case.

But at the same time the property MudAutocomplete<T>.ResetValueOnEmptyText property is left at the default value (false) which means that there will never be a null value.

In my opinion the MudBlazor project did not properly null-annotate their SetValueAsync method to allow for null values and so we can only wait for an update from them.

So to summarize:

I prefer reevaluating issue #105 as soon as https://github.com/MudBlazor/MudBlazor/issues/6535 is solved. Thanks again for your efforts!