You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on May 1, 2024. It is now read-only.
Right now all the code generators are written from scratch. As version 8 of CommunityToolkit.Mvvm is oficialy released it would be great to use it. We could use NotifyPropertyChangedFor, mesaging, and other goodnes which is going to come in future releases.
Code sample
Necessary change in ViewModelBase:
[INotifyPropertyChanged]
public abstract partial class ViewModelBase
{
and removal of INotifyPropertyChanged stuff as CommunityToolkit.Mvvm does it for us.
2. There is a need to change Set to SetProperty (so it introduces breaking change)
3. [Notify] must be changed to [ObservableProperty]
Thanks for the heads up, I already had a look at the release and I'm planning to implement this in the next major version. However, this will need a bigger refactoring of the whole library than just changing out some classes. The community toolkit has way more features than this library so they would need to be supported first.
Feature description
Right now all the code generators are written from scratch. As version 8 of CommunityToolkit.Mvvm is oficialy released it would be great to use it. We could use NotifyPropertyChangedFor, mesaging, and other goodnes which is going to come in future releases.
Code sample
and removal of INotifyPropertyChanged stuff as CommunityToolkit.Mvvm does it for us.
2. There is a need to change Set to SetProperty (so it introduces breaking change)
3. [Notify] must be changed to [ObservableProperty]
I created working repo:
https://github.com/alikoli/MvvmBlazor-with-CommunityToolkit.Mvvm
No response
The text was updated successfully, but these errors were encountered: