-
Notifications
You must be signed in to change notification settings - Fork 625
Upgrade .NET MAUI Workshop to .NET 10 #152
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
base: main
Are you sure you want to change the base?
Conversation
Co-authored-by: jamesmontemagno <1676321+jamesmontemagno@users.noreply.github.com>
|
@copilot We will want to tell people they will need .NET 10 and Visual Studio 2026 or Visual Studio Code with the C# Dev Kit. Visual Studio 2022 isn't supported with .NET 10 Also https://learn.microsoft.com/en-us/dotnet/maui/whats-new/dotnet-10?view=net-maui-10.0 for anythign deprecated we need to rmvoe at all |
… DisplayAlertAsync Co-authored-by: jamesmontemagno <1676321+jamesmontemagno@users.noreply.github.com>
Updated in commit 722f1af:
|
|
@copilot let's update the logging to 10.0.- Also update the community toolkit to 8.4.0, but take into consideration https://learn.microsoft.com/en-us/dotnet/communitytoolkit/mvvm/generators/errors/mvvmtk0045 and that we need to set LangVersion to preview in the csprj |
…ersion preview Co-authored-by: jamesmontemagno <1676321+jamesmontemagno@users.noreply.github.com>
Updated in commit 7ea2204:
|
|
@copilot We will want to update all viewmodels and all documentation to use the new mvvm toolkit style in 8.4.0 FROM: [ObservableProperty] To: do thi sin all projects and update all readmes |
…et-presentations/dotnet-maui-workshop into copilot/update-to-dotnet-10
…tial property syntax Co-authored-by: jamesmontemagno <1676321+jamesmontemagno@users.noreply.github.com>
Updated in commit 982c240:
|
| field = value; | ||
| OnPropertyChanged(); | ||
| // Also raise the IsNotBusy property changed | ||
| OnPropertyChanged(nameof(IsNotBusy)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@copilot dont' change the ones thatw e have today taht aren't using [ObservableProperty] for in the readme, we are showing the before/after using the toolkit
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed in commit 494d9f5. Reverted the manual implementation section (showing bool isBusy; and string title;) in both Chinese README files to preserve the before/after examples that demonstrate INotifyPropertyChanged before introducing the MVVM Toolkit.
Co-authored-by: jamesmontemagno <1676321+jamesmontemagno@users.noreply.github.com>
CardView now inherits from Border instead of Frame, enabling use of Border-specific styling and features. This may affect the appearance and layout of CardView components in the application.
…et-presentations/dotnet-maui-workshop into copilot/update-to-dotnet-10
- Update getter to use `isBusy` instead of `field` - Ensure setter updates `isBusy` and raises PropertyChanged events
Upgrade .NET MAUI Workshop to .NET 10
Based on the official .NET MAUI 9 to 10 upgrade instructions, all tasks have been completed:
Summary
All 12 .NET MAUI projects have been successfully updated from .NET 9 to .NET 10:
net9.0-*tonet10.0-*in all .csproj filesDisplayAlertcalls withDisplayAlertAsync(18 ViewModels updated)<LangVersion>preview</LangVersion>to all 12 .csproj files for C# 14 preview features support[ObservableProperty] bool isBusy;to[ObservableProperty] public partial bool IsBusy { get; set; }Original prompt
💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.