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

Support all .NET Framework versions? #55

Closed
FantasticFiasco opened this issue Oct 31, 2018 · 14 comments
Closed

Support all .NET Framework versions? #55

FantasticFiasco opened this issue Oct 31, 2018 · 14 comments
Labels
enhancement New feature or request

Comments

@FantasticFiasco
Copy link
Owner

FantasticFiasco commented Oct 31, 2018

Investigate whether we should support all framework versions. Lets see how Newtonsoft.Json is supporting different versions.

For more on specifically .NET Core 3:

@virzak
Copy link
Contributor

virzak commented Nov 2, 2018

As I tried to port my project to .NET Core 3.0, mvvm-dialogs was one of the many assemblies that did not target either .NET Core or .NET Standard, which prevented the porting.

@FantasticFiasco
Copy link
Owner Author

Hi virzak!

Yes, the upcoming .NET Core 3.0 is one of the environments we don't support, but let me see if I can get a grasp of the amount of work required to support it. Do you have any good documentation you can point me towards, helping me on my way?

@virzak
Copy link
Contributor

virzak commented Nov 18, 2018

You can get the preview SDK from here:
https://github.com/dotnet/core-sdk

And run dotnet new wpf to see what the new project looks like.

You need to make 3 changes:

  • Target is netcoreapp3.0 and
  • Add <UseWPF>true</UseWPF>
  • Add <Project Sdk="Microsoft.NET.Sdk.WindowsDesktop">

And it should compile right away.

@virzak
Copy link
Contributor

virzak commented Nov 18, 2018

And need to wait for multitargeting in order to not produce two versions of the library

@FantasticFiasco
Copy link
Owner Author

Thanks! I'll get right on it as soon as I finish what I'm currently working on. You ought to expect something in the upcoming week.

@FantasticFiasco FantasticFiasco added the enhancement New feature or request label Nov 26, 2018
@FantasticFiasco
Copy link
Owner Author

@virzak I've hacked up a pre-release preview of the .NET Core 3.0 support, you can find it here on nuget.org.

For now I'm satisfied. When .NET Core 3.0 finally ships we'll be able to let MvvmDialogs support it.

Best of lucks with validating your own application for .NET Core 3.0 support!

@FantasticFiasco
Copy link
Owner Author

.NET Core 3.0 is now released

@virzak
Copy link
Contributor

virzak commented Sep 24, 2019

Amazing!!

@FantasticFiasco
Copy link
Owner Author

I'm unfortunately on holiday this week and next, but will after that start the work to support .NET Core 3.0. I cant wait to get started

@FantasticFiasco
Copy link
Owner Author

I've now finally published a beta1 on NuGet. Would it be to asking to much if you where to try it out and determine whether it fits your needs?

@virzak
Copy link
Contributor

virzak commented Nov 1, 2019

@FantasticFiasco,

All dialogs in my biggest application that uses this library work right out of the box with the latest beta.

I briefly went through examples and discovered at least one bug. I can make a separate issue with suggestions over the weekend if you'd like. Will briefly mention here though:

The bug I found: in Demo.ActivateNonModalDialog.Core you can only show the time dialog once, because the dialogViewModel reference is never reset to null.

Recently I replaced SimpleIoc with Microsoft.Extensions.DependencyInjection since it plays better with Polly (Microsoft.Extensions.Http.Polly) and EF Core. In general there seems to be very little activity in mvvmlight repo, which is a shame cause it is awesome.

@FantasticFiasco
Copy link
Owner Author

All dialogs in my biggest application that uses this library work right out of the box with the latest beta.

Perfect, then I'll start the process of releasing an official version.

I can make a separate issue with suggestions over the weekend if you'd like.

I would definitely like that. When creating the new .NET Core samples, I noted that the can-execute function wasn't triggered as it does in .NET Framework. Or I made a mistake when I copied the samples and migrated them to .NET Core. Whatever it is, I am eager for your contribution. Thank you very much for your support!

Recently I replaced SimpleIoc with Microsoft.Extensions.DependencyInjection since it plays better with Polly (Microsoft.Extensions.Http.Polly) and EF Core.

I'll look into that IoC container. I opted to chose sample dependencies that would be familiar to new WPF developers. I'll look into the popularity of Microsoft.Extensions.DependencyInjection. If it gains traction over SimpleIoc I'll have no problem doing the switch.

@FantasticFiasco
Copy link
Owner Author

v6.0.0 is now officially released. I'll close this issue and if we find any problems we'll open new issues.

@FantasticFiasco
Copy link
Owner Author

The bug I found: in Demo.ActivateNonModalDialog.Core you can only show the time dialog once, because the dialogViewModel reference is never reset to null.

I think I've found another bug in the .NET Core samples. The command's can-execute method on some samples is not automatically invoked, as it is using MVVM Light in .NET Framework, but has to be invoked by me.

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

No branches or pull requests

2 participants