-
Notifications
You must be signed in to change notification settings - Fork 492
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
[VDG] UI Decoupling #53 #11801
[VDG] UI Decoupling #53 #11801
Conversation
WalletWasabi.Fluent/ViewModels/Wallets/Advanced/WalletCoins/WalletCoinsViewModel.cs
Outdated
Show resolved
Hide resolved
WalletWasabi.Fluent/ViewModels/Wallets/WalletSettingsViewModel.cs
Outdated
Show resolved
Hide resolved
I think you already removed the workaround 😁 The problem we had before this PR is that we were using and old version of DynamicData (the minimum version to satisfy other dependencies), so we were missing the latest features that were added lately. Some of them will greatly help us simplify/improve our code. As a side note and always my opinion, we should look to the DynamicData project with attention and try to be as skilled a possible. It always pays off!. It's one of the most powerful libraries that I've found out there. |
MainViewModel
DynamicData
to latest versionWalletModel.IsCoinjoining
bool property, as a workaround for missing DynamicData functionality. @SuperJMN can you comment on what's required to remove this?