Skip to content
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

ReactiveCommand is changed? #8

Open
neslekkim opened this issue Nov 14, 2018 · 1 comment
Open

ReactiveCommand is changed? #8

neslekkim opened this issue Nov 14, 2018 · 1 comment

Comments

@neslekkim
Copy link

Trying to learn, and downloaded this code, but after upgrading reactive and stuff, I get error in LogEntryViewer, that the ReactiveCommand is defined wrong?

Severity Code Description Project File Line Suppression State
Error CS1061 'ReactiveCommand' does not contain a definition for 'Dispose' and no accessible extension method 'Dispose' accepting a first argument of type 'ReactiveCommand' could be found (are you missing a using directive or an assembly reference?) Trader.Client LogEntryViewer.cs 91 Active
Error CS0029 Cannot implicitly convert type 'ReactiveUI.ReactiveCommand<System.Reactive.Unit, System.Reactive.Unit>' to 'ReactiveUI.ReactiveCommand' Trader.Client LogEntryViewer.cs 77 Active
Error CS0718 'ReactiveCommand': static types cannot be used as type arguments Trader.Client LogEntryView.xaml.cs 26 Active
Error CS0722 'ReactiveCommand': static types cannot be used as return types Trader.Client LogEntryViewer.cs 27 Active

Since I'm trying to learn this, I don't know what the error is though.. blush

@worldbeater
Copy link

ReactiveCommand abstract base class has been deprecated and removed in latest ReactiveUI releases, so need to change all occurrences of ReactiveCommand to ReactiveCommand<Unit, Unit> during the upgrade.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants