Feedback updates

Author: aritchie

OwnerModule/Handlers/DeleteEventHandlers.cs

@@ -12,7 +12,7 @@ public async Task Handle(DeleteVehicleEvent @event, CancellationToken cancellati
     {
         await data.DeleteByVehicle(@event.VehicleId, cancellationToken);
 
-        // this tells bulti-in mediator components to bust their cache values
+        // this tells built-in mediator components to bust their cache values
         await mediator.FlushAllStores(cancellationToken); 
     }
 

OwnerModule/LinkViewModel.cs

@@ -38,7 +38,7 @@ public override async void OnNavigatedTo(INavigationParameters parameters)
         base.OnNavigatedTo(parameters);
         if (parameters.IsNewNavigation())
         {
-            var request = parameters.Get<LinkNavRequest>();
+            var request = parameters.GetRequired<LinkNavRequest>();
             await Task.WhenAll(this.BindPeople(), this.BindVehicles());
 
             if (request.PersonId == null)

PeopleModule.Contracts/Routes.cs

@@ -2,6 +2,6 @@ namespace PeopleModule;
 
 public static class Routes
 {
-    public static string List = "People";
-    public static string Detail = "Person";
+    public const string List = "People";
+    public const string Detail = "Person";
 }

PeopleModule/DetailViewModel.cs

@@ -15,7 +15,7 @@ public override async void OnNavigatedTo(INavigationParameters parameters)
 
         if (parameters.IsNewNavigation())
         {
-            var request = parameters.Get<DetailNavRequest>()!;
+            var request = parameters.GetRequired<DetailNavRequest>();
             this.person = await data.GetById(request.PersonId, CancellationToken.None);
             this.Title = this.person!.FullName;
 

PeopleModule/ListPage.xaml

@@ -3,6 +3,7 @@
              xmlns:x="http://schemas.microsoft.com/winfx/2009/xaml"
              xmlns:peopleModule="clr-namespace:PeopleModule"
              xmlns:contracts="clr-namespace:PeopleModule.Contracts;assembly=PeopleModule.Contracts"
+             xmlns:skeleton="clr-namespace:Maui.Skeleton;assembly=Maui.Skeleton"
              NavigationPage.BackButtonTitle="Back"
              x:DataType="peopleModule:ListViewModel"
              x:Class="PeopleModule.ListPage"
@@ -12,7 +13,10 @@
                      IsRefreshing="{Binding IsBusy}">
             <CollectionView ItemsSource="{Binding List}"
                             SelectionMode="Single"
-                            SelectedItem="{Binding SelectedPerson}">
+                            SelectedItem="{Binding SelectedPerson}"
+                            skeleton:Skeleton.IsParent="True"
+                            skeleton:Skeleton.IsBusy="{Binding IsBusy}"
+                            skeleton:Skeleton.Animation="{skeleton:DefaultAnimation Fade}">
                 <CollectionView.EmptyView>
                     <Label Text="No People Found"
                            FontSize="Large" 

PeopleModule/PeopleModule.csproj

@@ -9,6 +9,7 @@
     <ItemGroup>
         <ProjectReference Include="..\SharedLib\SharedLib.csproj"/>
         <PackageReference Include="Bogus" Version="35.5.1"/>
+        <PackageReference Include="HorusStudio.Maui.Skeleton" Version="2.0.0" />
         <PackageReference Include="ReactiveUI.Fody" Version="19.5.41"/>
     </ItemGroup>
 </Project>

SharedLib/Extensions.cs

@@ -2,9 +2,12 @@ namespace SharedLib;
 
 public static class Extensions
 {
-    public static T? Get<T>(this INavigationParameters parameters)
-        => parameters.GetValue<T>(typeof(T).Name);
-
-    public static (string, TRequest) ToNavParam<TRequest>(this TRequest request) where TRequest : IRequest
-        => (request.GetType().Name, request);
+    public static T GetRequired<T>(this INavigationParameters parameters)
+    {
+        var key = typeof(T).Name;
+        if (!parameters.ContainsKey(key))
+            throw new InvalidOperationException($"NavParameter '{key}' was not found");
+        
+        return parameters.GetValue<T>(key);
+    }
 }

VehicleModule.Contracts/Routes.cs

@@ -2,6 +2,6 @@ namespace VehicleModule;
 
 public static class Routes
 {
-    public static string List = "Animals";
-    public static string Detail = "Animal";
+    public const string List = "Animals";
+    public const string Detail = "Animal";
 }

VehicleModule/DetailViewModel.cs

@@ -15,10 +15,11 @@ IMediator mediator
     public override async void OnNavigatedTo(INavigationParameters parameters)
     {
         base.OnNavigatedTo(parameters);
-        var request = parameters.Get<DetailNavRequest>()!;
 
         if (parameters.IsNewNavigation())
         {
+            var request = parameters.GetRequired<DetailNavRequest>();
+            
             // go idea to safety this
             this.vehicle = await data.GetById(request.VehicleId, CancellationToken.None);
             this.Title = this.vehicle!.Name;

VehicleModule/ListPage.xaml

@@ -3,6 +3,7 @@
              xmlns:x="http://schemas.microsoft.com/winfx/2009/xaml"
              xmlns:vehicleModule="clr-namespace:VehicleModule"
              xmlns:contracts="clr-namespace:VehicleModule.Contracts;assembly=VehicleModule.Contracts"
+             xmlns:skeleton="clr-namespace:Maui.Skeleton;assembly=Maui.Skeleton"
              NavigationPage.BackButtonTitle="Back"
              x:DataType="vehicleModule:ListViewModel"
              x:Class="VehicleModule.ListPage"
@@ -12,7 +13,10 @@
                      IsRefreshing="{Binding IsBusy}">
             <CollectionView ItemsSource="{Binding List}"
                             SelectionMode="Single"
-                            SelectedItem="{Binding SelectedVehicle}">
+                            SelectedItem="{Binding SelectedVehicle}"
+                            skeleton:Skeleton.IsParent="True"
+                            skeleton:Skeleton.IsBusy="{Binding IsBusy}"
+                            skeleton:Skeleton.Animation="{skeleton:DefaultAnimation Fade}">
                 <CollectionView.EmptyView>
                     <Label Text="No Vehicles Found"
                            FontSize="Large"