-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Fixes MudDataGrid NullReferenceExeption #6541
Conversation
Fixes #6529 (MudDataGrid NullReferenceException when using PropertyColumn with formatting)
@henon @tjscience would be nice to merge this asap. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Perfect PR, Thanks!
@HClausing is the issue you reported fixed by this too? |
I didn't test this branch directly but looking at the code I'm sure it will work as expected. Is there a dev release for testing? |
Here you go: https://www.nuget.org/packages/MudBlazor/6.2.1-dev1 |
@henon v6.2.1-dev1 nuget shows "False" on all my mudtables. |
hmm, did I do something wrong in the build process? I'll clean and try again. Don't see any of those on https://dev.mudblazor.com/components/table |
I suspect this: bace647#r106458878 |
Fixed. |
Made a new pre-release covering recently fixed DataGrid regressions: https://www.nuget.org/packages/MudBlazor/6.2.1-dev2 |
* Update MudDataGrid.razor Fixes MudBlazor#6529 (MudDataGrid NullReferenceException when using PropertyColumn with formatting) * Tests added
Description
Fixes MudDataGrid NullReferenceExeption when rendering a null if the PropertyColumn has formatting
Fixes #6529
How Has This Been Tested?
Unit tests and visually.
Types of changes
Checklist:
dev
).