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

Avalonia version choice for MVVM templates #135

Merged

Conversation

Mrxx99
Copy link
Contributor

@Mrxx99 Mrxx99 commented Dec 3, 2022

This adds also Avalonia version choice to the MVVM templates and simplifies the choosing for all.
With the old way it would be nested #ifs for the MVVM templates.
With the new way the version has also now only to be specified for the choices and not again in the csproj files.

If the simplified way is not liked I can also revert to the old #if/ #elseif syntax, so only the change that the Avalonia version is added to the MVVM templates remains (first commit).

@maxkatz6 maxkatz6 requested a review from Takoooooo December 7, 2022 05:19
Copy link
Contributor

@Takoooooo Takoooooo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM,thanks for doing that,it really helps!

@Takoooooo
Copy link
Contributor

If you are happy to proceed helping us with this i may create a project board where i will note everything that we need to do to make our templates fully usable from VS without extension.

@Takoooooo Takoooooo merged commit 6be16ee into AvaloniaUI:master Dec 7, 2022
@Mrxx99
Copy link
Contributor Author

Mrxx99 commented Dec 10, 2022

@Takoooooo Yes, I am happy to proceed helping here, the board sound like a good idea.

@Mrxx99 Mrxx99 deleted the feature/avalonia-version-for-mvvm branch January 1, 2023 21:38
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

Successfully merging this pull request may close these issues.

3 participants