From 8b86007ca75b2a1062fca40656a722021a445128 Mon Sep 17 00:00:00 2001
From: Andrii Snihyr <berserkerdotnet@gmail.com>
Date: Sat, 20 May 2023 10:50:46 -0700
Subject: [PATCH] Move to .NET 8 and minor fixes (#9)

* fix width of the set indicator
fix persistance of the model when sets are updated
migrate to .NET 8

* fix width of the set indicator
fix persistance of the model when sets are updated
migrate to .NET 8

* run on windows

* package updates and using a flag

* no framework flag

* downgrade package due to https://github.com/xamarin/AndroidX/issues/717
https://github.com/xamarin/GooglePlayServicesComponents/issues/694
---
 .github/workflows/publish-wt-service.yml      |  4 +-
 .github/workflows/workout-tracker-api.yml     |  6 +-
 build/build.csproj                            |  2 +-
 .../Controls/ExerciseSetView.xaml             |  4 +-
 .../Controls/ExerciseSetsCollection.xaml      |  2 +-
 .../Controls/ExerciseSetsCollection.xaml.cs   |  6 +-
 .../Controls/WorkoutDefinitionEditor.xaml     |  2 +-
 .../Extensions/ServiceCollectionExtensions.cs | 33 +--------
 .../Services/ApiRepositoryClient.cs           |  1 +
 .../Services/Data/DataSyncWorker.cs           |  1 +
 .../ViewModels/EditExercisePageViewModel.cs   |  1 +
 src/WorkoutTracker.MAUI/Views/Workout.xaml    |  5 +-
 src/WorkoutTracker.MAUI/Views/Workout.xaml.cs |  2 +-
 .../WorkoutTracker.MAUI.csproj                | 25 +++----
 .../WorkoutTracker.Models.csproj              |  2 +-
 .../ApplicationContext.cs                     |  2 +-
 .../Extensions/IExerciseSetExtensions.cs      | 34 +++++++++
 .../Interfaces/IWorkoutDataProvider.cs        |  2 +
 .../Models/SetWrapper.cs                      | 10 +++
 .../ViewModels/WorkoutViewModel.cs            | 29 ++++----
 .../WorkoutTracker.Services.csproj            |  2 +-
 .../ViewModels/WorkoutViewModelTests.cs       | 74 +++++++++++++++++++
 .../WorkoutTracker.UnitTests.csproj           |  2 +-
 23 files changed, 169 insertions(+), 82 deletions(-)
 rename src/{WorkoutTracker.MAUI/Services => WorkoutTracker.Services}/ApplicationContext.cs (96%)
 create mode 100644 src/WorkoutTracker.Services/Extensions/IExerciseSetExtensions.cs
 create mode 100644 src/WorkoutTracker.Services/Models/SetWrapper.cs
 rename src/{WorkoutTracker.MAUI => WorkoutTracker.Services}/ViewModels/WorkoutViewModel.cs (92%)
 create mode 100644 tests/WorkoutTracker.UnitTests/ViewModels/WorkoutViewModelTests.cs

diff --git a/.github/workflows/publish-wt-service.yml b/.github/workflows/publish-wt-service.yml
index f3985df..43fc51d 100644
--- a/.github/workflows/publish-wt-service.yml
+++ b/.github/workflows/publish-wt-service.yml
@@ -10,7 +10,7 @@ env:
   PROJECT_NAME_FOR_DOCKER: workouttracker-api
 jobs:
   publish:
-    runs-on: ubuntu-latest
+    runs-on: windows-latest
     steps:
     - name: Checkout to the branch
       uses: actions/checkout@v2
@@ -34,7 +34,7 @@ jobs:
         name: binlog
         path: msbuild.binlog
   deploy:
-    runs-on: ubuntu-latest
+    runs-on: windows-latest
     needs: publish
     steps:
     - name: Azure Login
diff --git a/.github/workflows/workout-tracker-api.yml b/.github/workflows/workout-tracker-api.yml
index ba138c6..20fa7da 100644
--- a/.github/workflows/workout-tracker-api.yml
+++ b/.github/workflows/workout-tracker-api.yml
@@ -10,11 +10,11 @@ env:
   CONTAINER_APP_NAME: workout-tracker-api
   CONTAINER_APP_RESOURCE_GROUP_NAME: workout-tracker
   CONTAINER_REGISTRY_LOGIN_SERVER: berserkerdotnetcregistry.azurecr.io
-  DOTNET_CORE_VERSION: 7.0.x
+  DOTNET_CORE_VERSION: 8.0.x
   PROJECT_NAME_FOR_DOCKER: workouttracker-api
 jobs:
   build:
-    runs-on: ubuntu-latest
+    runs-on: windows-latest
     steps:
     - name: Checkout to the branch
       uses: actions/checkout@v2
@@ -30,7 +30,7 @@ jobs:
     - name: Install dependencies
       run: dotnet restore
     - name: Build
-      run: dotnet build --configuration Release --no-restore -bl
+      run: dotnet build --configuration Release --no-restore -bl -p:AndroidEnableMarshalMethods=false
     - name: Test
       run: dotnet test --no-restore --verbosity normal
     - name: Upload binlog for investigation
diff --git a/build/build.csproj b/build/build.csproj
index ae21513..3cddce0 100644
--- a/build/build.csproj
+++ b/build/build.csproj
@@ -2,7 +2,7 @@
 
   <PropertyGroup>
     <OutputType>Exe</OutputType>
-    <TargetFramework>net7.0</TargetFramework>
+    <TargetFramework>net8.0</TargetFramework>
     <RootNamespace></RootNamespace>
     <NoWarn>CS0649;CS0169</NoWarn>
     <NukeRootDirectory>..</NukeRootDirectory>
diff --git a/src/WorkoutTracker.MAUI/Controls/ExerciseSetView.xaml b/src/WorkoutTracker.MAUI/Controls/ExerciseSetView.xaml
index a9f00fe..2017e18 100644
--- a/src/WorkoutTracker.MAUI/Controls/ExerciseSetView.xaml
+++ b/src/WorkoutTracker.MAUI/Controls/ExerciseSetView.xaml
@@ -2,7 +2,7 @@
 <ContentView xmlns="http://schemas.microsoft.com/dotnet/2021/maui"
              xmlns:x="http://schemas.microsoft.com/winfx/2009/xaml"
              xmlns:dxe="clr-namespace:DevExpress.Maui.Editors;assembly=DevExpress.Maui.Editors"
-             xmlns:ctrl="clr-namespace:WorkoutTracker.MAUI.Controls"
+             xmlns:ctrl="clr-namespace:WorkoutTracker.Services.Models;assembly=WorkoutTracker.Services"
              x:Class="WorkoutTracker.MAUI.Controls.ExerciseSetView">
     <Grid Margin="0,0,8,0" x:DataType="ctrl:SetWrapper">
         <Grid.ColumnDefinitions>
@@ -10,7 +10,7 @@
             <ColumnDefinition Width="*" />
             <ColumnDefinition Width="*" />
         </Grid.ColumnDefinitions>
-        <dxe:Chip Text="{Binding Number}" BackgroundColor="{Binding Color}" Grid.Column="0" HeightRequest="40" WidthRequest="40" Tap="OnTap" />
+        <dxe:Chip Text="{Binding Number}" BackgroundColor="{Binding Color}" Grid.Column="0" HeightRequest="50" WidthRequest="50" Tap="OnTap" />
         <dxe:NumericEdit Value="{Binding Set.Weight, Mode=TwoWay}" Prefix="Weight:" Suffix="LB" Grid.Column="1" BoxMode="Filled" BackgroundColor="Transparent" MinValue="0" SelectValueOnFocus="True" />
         <dxe:NumericEdit Value="{Binding Set.Repetitions}" Prefix="Reps:" Grid.Column="2" BoxMode="Filled" BackgroundColor="Transparent" MinValue="0" SelectValueOnFocus="True" />
     </Grid>
diff --git a/src/WorkoutTracker.MAUI/Controls/ExerciseSetsCollection.xaml b/src/WorkoutTracker.MAUI/Controls/ExerciseSetsCollection.xaml
index 95b0d7d..d1def39 100644
--- a/src/WorkoutTracker.MAUI/Controls/ExerciseSetsCollection.xaml
+++ b/src/WorkoutTracker.MAUI/Controls/ExerciseSetsCollection.xaml
@@ -62,7 +62,7 @@
                     x:Name="setsStatus"
                     Margin="10,2"
                     HeightRequest="40"
-                    WidthRequest="50" />
+                    WidthRequest="60" />
             </StackLayout>
             <dxe:Chip x:Name="contextMenuBtn"
                       Grid.Column="1"
diff --git a/src/WorkoutTracker.MAUI/Controls/ExerciseSetsCollection.xaml.cs b/src/WorkoutTracker.MAUI/Controls/ExerciseSetsCollection.xaml.cs
index 47d7442..cf74708 100644
--- a/src/WorkoutTracker.MAUI/Controls/ExerciseSetsCollection.xaml.cs
+++ b/src/WorkoutTracker.MAUI/Controls/ExerciseSetsCollection.xaml.cs
@@ -7,6 +7,7 @@
 using WorkoutTracker.Models.Contracts;
 using WorkoutTracker.Models.Entities;
 using WorkoutTracker.Models.Presentation;
+using WorkoutTracker.Services.Models;
 
 namespace WorkoutTracker.MAUI.Controls;
 
@@ -205,9 +206,4 @@ private void OnReduceSetsClicked(object sender, System.EventArgs e)
             ReduceSetsCommand.Execute(_viewModel);
         }
     }
-}
-
-public record SetWrapper(int Number, IExerciseSet Set, LogEntryViewModel Model)
-{
-    public Color Color => Set.GetColor();
 }
\ No newline at end of file
diff --git a/src/WorkoutTracker.MAUI/Controls/WorkoutDefinitionEditor.xaml b/src/WorkoutTracker.MAUI/Controls/WorkoutDefinitionEditor.xaml
index 04bad17..ee923e1 100644
--- a/src/WorkoutTracker.MAUI/Controls/WorkoutDefinitionEditor.xaml
+++ b/src/WorkoutTracker.MAUI/Controls/WorkoutDefinitionEditor.xaml
@@ -2,7 +2,7 @@
 <ContentView xmlns="http://schemas.microsoft.com/dotnet/2021/maui"
              xmlns:x="http://schemas.microsoft.com/winfx/2009/xaml"
              xmlns:dxe="clr-namespace:DevExpress.Maui.Editors;assembly=DevExpress.Maui.Editors"
-             xmlns:ctrl="clr-namespace:WorkoutTracker.MAUI.Controls"
+             xmlns:ctrl="clr-namespace:WorkoutTracker.Services.Models;assembly=WorkoutTracker.Services"
              x:Class="WorkoutTracker.MAUI.Controls.WorkoutDefinitionEditor">
     <Grid Margin="0,0,8,0" x:DataType="ctrl:SetWrapper">
         <Grid.ColumnDefinitions>
diff --git a/src/WorkoutTracker.MAUI/Extensions/ServiceCollectionExtensions.cs b/src/WorkoutTracker.MAUI/Extensions/ServiceCollectionExtensions.cs
index 73f882a..3db393c 100644
--- a/src/WorkoutTracker.MAUI/Extensions/ServiceCollectionExtensions.cs
+++ b/src/WorkoutTracker.MAUI/Extensions/ServiceCollectionExtensions.cs
@@ -1,11 +1,7 @@
 using Microsoft.Extensions.DependencyInjection;
-using Microsoft.Maui.Graphics;
 using Microsoft.Maui.Hosting;
 using WorkoutTracker.MAUI.ViewModels;
 using WorkoutTracker.MAUI.Views;
-using WorkoutTracker.Models.Contracts;
-using WorkoutTracker.Models.Entities;
-using WorkoutTracker.Models.Presentation;
 using WorkoutTracker.Services.ViewModels;
 
 namespace WorkoutTracker.MAUI.Extensions;
@@ -33,31 +29,4 @@ public static MauiAppBuilder AddViews(this MauiAppBuilder builder)
         builder.Services.AddSingleton<EditWorkoutDefinition>();
         return builder;
     }
-}
-
-public static class IExerciseSetExtensions
-{
-    public static Color GetColor(this IExerciseSet set)
-    {
-        return set switch
-        {
-            LegacySet => Colors.LightGreen,
-            CompletedSet => Colors.LightGreen,
-            InProgressSet => Colors.LightPink,
-            ProposedSet => Colors.LightSkyBlue,
-            _ => Colors.Gray
-        };
-    }
-
-    public static SetStatus GetStatus(this IExerciseSet set)
-    {
-        return set switch
-        {
-            LegacySet => SetStatus.Completed,
-            CompletedSet => SetStatus.Completed,
-            InProgressSet => SetStatus.InProgress,
-            ProposedSet => SetStatus.NotStarted,
-            _ => SetStatus.NotStarted
-        };
-    }
-}
+}
\ No newline at end of file
diff --git a/src/WorkoutTracker.MAUI/Services/ApiRepositoryClient.cs b/src/WorkoutTracker.MAUI/Services/ApiRepositoryClient.cs
index fea11ab..5961a97 100644
--- a/src/WorkoutTracker.MAUI/Services/ApiRepositoryClient.cs
+++ b/src/WorkoutTracker.MAUI/Services/ApiRepositoryClient.cs
@@ -14,6 +14,7 @@
 using WorkoutTracker.Models.Contracts;
 using WorkoutTracker.Models.Entities;
 using WorkoutTracker.Models.Presentation;
+using WorkoutTracker.Services;
 
 namespace WorkoutTracker.MAUI.Services;
 
diff --git a/src/WorkoutTracker.MAUI/Services/Data/DataSyncWorker.cs b/src/WorkoutTracker.MAUI/Services/Data/DataSyncWorker.cs
index 4a8c51d..03d06d3 100644
--- a/src/WorkoutTracker.MAUI/Services/Data/DataSyncWorker.cs
+++ b/src/WorkoutTracker.MAUI/Services/Data/DataSyncWorker.cs
@@ -16,6 +16,7 @@
 using WorkoutTracker.MAUI.Services.Data.Entities;
 using WorkoutTracker.Models.Entities;
 using WorkoutTracker.Models.Presentation;
+using WorkoutTracker.Services;
 using Xamarin.Android.Net;
 
 namespace WorkoutTracker.MAUI.Services.Data;
diff --git a/src/WorkoutTracker.MAUI/ViewModels/EditExercisePageViewModel.cs b/src/WorkoutTracker.MAUI/ViewModels/EditExercisePageViewModel.cs
index 33d7386..d272c69 100644
--- a/src/WorkoutTracker.MAUI/ViewModels/EditExercisePageViewModel.cs
+++ b/src/WorkoutTracker.MAUI/ViewModels/EditExercisePageViewModel.cs
@@ -4,6 +4,7 @@
 using WorkoutTracker.MAUI.Services;
 using WorkoutTracker.MAUI.Services.Data;
 using WorkoutTracker.Models.Presentation;
+using WorkoutTracker.Services;
 
 namespace WorkoutTracker.MAUI.ViewModels;
 
diff --git a/src/WorkoutTracker.MAUI/Views/Workout.xaml b/src/WorkoutTracker.MAUI/Views/Workout.xaml
index 404d910..c39d896 100644
--- a/src/WorkoutTracker.MAUI/Views/Workout.xaml
+++ b/src/WorkoutTracker.MAUI/Views/Workout.xaml
@@ -3,7 +3,7 @@
              xmlns:x="http://schemas.microsoft.com/winfx/2009/xaml"
              xmlns:conv="clr-namespace:WorkoutTracker.MAUI.Converters"
              xmlns:ctrl="clr-namespace:WorkoutTracker.MAUI.Controls"
-             xmlns:vm="clr-namespace:WorkoutTracker.MAUI.ViewModels"
+             xmlns:vm="clr-namespace:WorkoutTracker.Services.ViewModels;assembly=WorkoutTracker.Services"
              xmlns:model="clr-namespace:WorkoutTracker.Models.Presentation;assembly=WorkoutTracker.Models"
              xmlns:entities="clr-namespace:WorkoutTracker.Models.Entities;assembly=WorkoutTracker.Models"
              xmlns:dxcv="clr-namespace:DevExpress.Maui.CollectionView;assembly=DevExpress.Maui.CollectionView"
@@ -50,7 +50,8 @@
             Icon="timer"
             IconVisible="True"
             IconColor="White" />
-        <Label Text="No exercises for today." HorizontalOptions="CenterAndExpand" HorizontalTextAlignment="Center" Margin="8,0,0,0" IsVisible="{Binding TodaySets, Converter={StaticResource CollectionToVisibilityConverter}}" Grid.Row="1" />
+        <Label Text="No exercises for today." HorizontalOptions="CenterAndExpand" HorizontalTextAlignment="Center" Margin="8,0,0,0"
+               IsVisible="{Binding TodaySets, Converter={StaticResource CollectionToVisibilityConverter}}" Grid.Row="1" />
         <dxcv:DXCollectionView
                 x:Name="setsList"
                 Grid.Row="1"
diff --git a/src/WorkoutTracker.MAUI/Views/Workout.xaml.cs b/src/WorkoutTracker.MAUI/Views/Workout.xaml.cs
index 3b4c484..fc010a8 100644
--- a/src/WorkoutTracker.MAUI/Views/Workout.xaml.cs
+++ b/src/WorkoutTracker.MAUI/Views/Workout.xaml.cs
@@ -1,7 +1,7 @@
 using AndroidX.Work;
 using System;
 using WorkoutTracker.MAUI.Services.Data;
-using WorkoutTracker.MAUI.ViewModels;
+using WorkoutTracker.Services.ViewModels;
 
 namespace WorkoutTracker.MAUI.Views;
 
diff --git a/src/WorkoutTracker.MAUI/WorkoutTracker.MAUI.csproj b/src/WorkoutTracker.MAUI/WorkoutTracker.MAUI.csproj
index 7795fa4..b498efe 100644
--- a/src/WorkoutTracker.MAUI/WorkoutTracker.MAUI.csproj
+++ b/src/WorkoutTracker.MAUI/WorkoutTracker.MAUI.csproj
@@ -1,6 +1,6 @@
 <Project Sdk="Microsoft.NET.Sdk">
 	<PropertyGroup>
-    <TargetFramework>net7.0-android</TargetFramework>
+    <TargetFramework>net8.0-android</TargetFramework>
 		<OutputType>Exe</OutputType>
 		<SingleProject>true</SingleProject>
 		<UseMaui>true</UseMaui>
@@ -18,9 +18,9 @@
 		<ApplicationVersion>1</ApplicationVersion>
 		<ApplicationDisplayVersion>2.0</ApplicationDisplayVersion>
 
-		<UseInterpreter Condition="'$(TargetFramework)' == 'net7.0-android'">False</UseInterpreter>
+		<UseInterpreter Condition="'$(TargetFramework)' == 'net8.0-android'">False</UseInterpreter>
 		<LangVersion>preview</LangVersion>
-		<SupportedOSPlatformVersion Condition="'$(TargetFramework)' == 'net7.0-android'">28.0</SupportedOSPlatformVersion>
+		<SupportedOSPlatformVersion Condition="'$(TargetFramework)' == 'net8.0-android'">28.0</SupportedOSPlatformVersion>
 	</PropertyGroup>
 
 	<ItemGroup>
@@ -64,23 +64,22 @@
 	</ItemGroup>
 
 	<ItemGroup>
-    <PackageReference Include="Plugin.LocalNotification" Version="10.0.3" />
-    <PackageReference Include="CommunityToolkit.Maui" Version="3.1.0" />
+    <PackageReference Include="Plugin.LocalNotification" Version="10.1.4" />
+    <PackageReference Include="CommunityToolkit.Maui" Version="5.1.0" />
     <PackageReference Include="Xamarin.AndroidX.Work.Runtime" Version="2.7.1.5" />
-    <PackageReference Include="Xamarin.AndroidX.Lifecycle.LiveData" Version="2.5.1.1" />
-		<PackageReference Include="Microsoft.Extensions.FileProviders.Embedded" Version="7.0.0-rc.2.*" />
-		<PackageReference Include="Microsoft.Extensions.Http" Version="7.0.0-rc.2.*" />
-		<PackageReference Include="Microsoft.Identity.Client" Version="4.47.2" />
-		<PackageReference Include="System.IdentityModel.Tokens.Jwt" Version="6.25.0" />
-		<PackageReference Include="Microsoft.Extensions.Configuration.Json" Version="7.0.0-rc.2.*" />
-    <PackageReference Include="Microsoft.Extensions.Logging.Debug" Version="7.0.0-rc.2.*" />
+		<PackageReference Include="Microsoft.Extensions.FileProviders.Embedded" Version="8.0.0-preview.3.23177.8" />
+		<PackageReference Include="Microsoft.Extensions.Http" Version="8.0.0-preview.3.23174.8" />
+		<PackageReference Include="Microsoft.Identity.Client" Version="4.54.0" />
+		<PackageReference Include="System.IdentityModel.Tokens.Jwt" Version="6.30.1" />
+		<PackageReference Include="Microsoft.Extensions.Configuration.Json" Version="8.0.0-preview.3.23174.8" />
+    <PackageReference Include="Microsoft.Extensions.Logging.Debug" Version="8.0.0-preview.3.23174.8" />
     <PackageReference Include="DevExpress.Maui.Charts" Version="22.2.3" />
     <PackageReference Include="DevExpress.Maui.CollectionView" Version="22.2.3" />
     <PackageReference Include="DevExpress.Maui.Core" Version="22.2.3" />
     <PackageReference Include="DevExpress.Maui.DataGrid" Version="22.2.3" />
     <PackageReference Include="DevExpress.Maui.Editors" Version="22.2.3" />
     <PackageReference Include="DevExpress.Maui.Scheduler" Version="22.2.3" />
-    <PackageReference Include="CommunityToolkit.Mvvm" Version="8.0.0" />
+    <PackageReference Include="CommunityToolkit.Mvvm" Version="8.2.0" />
     <PackageReference Include="UnitsNet" Version="4.149.0" />
     <PackageReference Include="sqlite-net-pcl" Version="1.8.116" />
     <PackageReference Include="SQLitePCLRaw.bundle_green" Version="2.1.2" />
diff --git a/src/WorkoutTracker.Models/WorkoutTracker.Models.csproj b/src/WorkoutTracker.Models/WorkoutTracker.Models.csproj
index 18c6426..1b658aa 100644
--- a/src/WorkoutTracker.Models/WorkoutTracker.Models.csproj
+++ b/src/WorkoutTracker.Models/WorkoutTracker.Models.csproj
@@ -1,7 +1,7 @@
 <Project Sdk="Microsoft.NET.Sdk">
 
   <PropertyGroup>
-    <TargetFramework>net7.0</TargetFramework>
+    <TargetFrameworks>net7.0;net8.0</TargetFrameworks>
   </PropertyGroup>
 
   <ItemGroup>
diff --git a/src/WorkoutTracker.MAUI/Services/ApplicationContext.cs b/src/WorkoutTracker.Services/ApplicationContext.cs
similarity index 96%
rename from src/WorkoutTracker.MAUI/Services/ApplicationContext.cs
rename to src/WorkoutTracker.Services/ApplicationContext.cs
index 62fcbdb..b0fa5c5 100644
--- a/src/WorkoutTracker.MAUI/Services/ApplicationContext.cs
+++ b/src/WorkoutTracker.Services/ApplicationContext.cs
@@ -2,7 +2,7 @@
 using System;
 using WorkoutTracker.MAUI.Interfaces;
 
-namespace WorkoutTracker.MAUI.Services;
+namespace WorkoutTracker.Services;
 
 public class ApplicationContext : ILogger, INotificationService
 {
diff --git a/src/WorkoutTracker.Services/Extensions/IExerciseSetExtensions.cs b/src/WorkoutTracker.Services/Extensions/IExerciseSetExtensions.cs
new file mode 100644
index 0000000..4f16628
--- /dev/null
+++ b/src/WorkoutTracker.Services/Extensions/IExerciseSetExtensions.cs
@@ -0,0 +1,34 @@
+using Microsoft.Maui.Graphics;
+using WorkoutTracker.Models.Contracts;
+using WorkoutTracker.Models.Entities;
+using WorkoutTracker.Models.Presentation;
+
+namespace WorkoutTracker.Services.Extensions
+{
+    public static class IExerciseSetExtensions
+    {
+        public static Color GetColor(this IExerciseSet set)
+        {
+            return set switch
+            {
+                LegacySet => Colors.LightGreen,
+                CompletedSet => Colors.LightGreen,
+                InProgressSet => Colors.LightPink,
+                ProposedSet => Colors.LightSkyBlue,
+                _ => Colors.Gray
+            };
+        }
+
+        public static SetStatus GetStatus(this IExerciseSet set)
+        {
+            return set switch
+            {
+                LegacySet => SetStatus.Completed,
+                CompletedSet => SetStatus.Completed,
+                InProgressSet => SetStatus.InProgress,
+                ProposedSet => SetStatus.NotStarted,
+                _ => SetStatus.NotStarted
+            };
+        }
+    }
+}
\ No newline at end of file
diff --git a/src/WorkoutTracker.Services/Interfaces/IWorkoutDataProvider.cs b/src/WorkoutTracker.Services/Interfaces/IWorkoutDataProvider.cs
index 8ef10db..d5b8d04 100644
--- a/src/WorkoutTracker.Services/Interfaces/IWorkoutDataProvider.cs
+++ b/src/WorkoutTracker.Services/Interfaces/IWorkoutDataProvider.cs
@@ -23,4 +23,6 @@ public interface IWorkoutDataProvider
     Profile GetProfile();
 
     IEnumerable<WorkoutProgram> GetPrograms();
+
+    IEnumerable<LogEntryViewModel> GetTodaysSchedule();
 }
\ No newline at end of file
diff --git a/src/WorkoutTracker.Services/Models/SetWrapper.cs b/src/WorkoutTracker.Services/Models/SetWrapper.cs
new file mode 100644
index 0000000..e725b39
--- /dev/null
+++ b/src/WorkoutTracker.Services/Models/SetWrapper.cs
@@ -0,0 +1,10 @@
+using WorkoutTracker.Models.Contracts;
+using WorkoutTracker.Models.Presentation;
+using WorkoutTracker.Services.Extensions;
+
+namespace WorkoutTracker.Services.Models;
+
+public record SetWrapper(int Number, IExerciseSet Set, LogEntryViewModel Model)
+{
+    public Color Color => Set.GetColor();
+}
\ No newline at end of file
diff --git a/src/WorkoutTracker.MAUI/ViewModels/WorkoutViewModel.cs b/src/WorkoutTracker.Services/ViewModels/WorkoutViewModel.cs
similarity index 92%
rename from src/WorkoutTracker.MAUI/ViewModels/WorkoutViewModel.cs
rename to src/WorkoutTracker.Services/ViewModels/WorkoutViewModel.cs
index e5b023c..2c7146b 100644
--- a/src/WorkoutTracker.MAUI/ViewModels/WorkoutViewModel.cs
+++ b/src/WorkoutTracker.Services/ViewModels/WorkoutViewModel.cs
@@ -1,26 +1,20 @@
-using CommunityToolkit.Mvvm.Input;
-using DevExpress.Maui.Scheduler.Internal;
-using System;
-using System.Collections.Generic;
+using CommunityToolkit.Mvvm.ComponentModel;
+using CommunityToolkit.Mvvm.Input;
 using System.Collections.ObjectModel;
-using System.Linq;
-using System.Threading.Tasks;
-using WorkoutTracker.MAUI.Controls;
-using WorkoutTracker.MAUI.Extensions;
 using WorkoutTracker.MAUI.Interfaces;
-using WorkoutTracker.MAUI.Services;
-using WorkoutTracker.MAUI.Services.Data;
 using WorkoutTracker.Models.Contracts;
 using WorkoutTracker.Models.Entities;
 using WorkoutTracker.Models.Presentation;
 using WorkoutTracker.Models.Selectors;
-using WorkoutTracker.Services;
+using WorkoutTracker.Services.Extensions;
+using WorkoutTracker.Services.Interfaces;
+using WorkoutTracker.Services.Models;
 
-namespace WorkoutTracker.MAUI.ViewModels;
+namespace WorkoutTracker.Services.ViewModels;
 
 public sealed partial class WorkoutViewModel : ObservableObject
 {
-    private readonly WorkoutTrackerDb _trackerDb;
+    private readonly IWorkoutDataProvider _trackerDb;
     private readonly SetsGenerator _setsGenerator;
     private readonly IExerciseTimerService _timer;
     private readonly ApplicationContext<WorkoutViewModel> _context;
@@ -52,7 +46,7 @@ public sealed partial class WorkoutViewModel : ObservableObject
     [ObservableProperty]
     private ObservableCollection<ExerciseViewModel> _exercises;
 
-    public WorkoutViewModel(WorkoutTrackerDb trackerDb, SetsGenerator setsGenerator, IExerciseTimerService timer, ApplicationContext<WorkoutViewModel> context)
+    public WorkoutViewModel(IWorkoutDataProvider trackerDb, SetsGenerator setsGenerator, IExerciseTimerService timer, ApplicationContext<WorkoutViewModel> context)
     {
         _trackerDb = trackerDb;
         _setsGenerator = setsGenerator;
@@ -109,7 +103,10 @@ public void GetOrCreateWorkout()
                 .ToArray();
                 // Remove reserved from the list
 
-                reservedExercises.ForEach(r => exercises.Remove(r));
+                foreach (var reservedExercise in reservedExercises)
+                {
+                    exercises.Remove(reservedExercise);
+                }
                 foreach (var exerciseDefinition in todayWorkoutDefinition.Exercises)
                 {
                     // This sucks!
@@ -185,6 +182,7 @@ public void AddSet(LogEntryViewModel model)
     {
         var sets = model.Sets as ObservableCollection<IExerciseSet>;
         sets.Add(new ProposedSet { Repetitions = 0, Weight = 0 });
+        _trackerDb.UpdateViewModel(model);
     }
 
     [RelayCommand]
@@ -194,6 +192,7 @@ public void ReduceSets(LogEntryViewModel model)
         if (sets.Last() is ProposedSet)
         {
             sets.Remove(sets.Last());
+            _trackerDb.UpdateViewModel(model);
         }
     }
 
diff --git a/src/WorkoutTracker.Services/WorkoutTracker.Services.csproj b/src/WorkoutTracker.Services/WorkoutTracker.Services.csproj
index e305dd2..63ef42d 100644
--- a/src/WorkoutTracker.Services/WorkoutTracker.Services.csproj
+++ b/src/WorkoutTracker.Services/WorkoutTracker.Services.csproj
@@ -1,7 +1,7 @@
 <Project Sdk="Microsoft.NET.Sdk">
 
   <PropertyGroup>
-    <TargetFrameworks>net7.0;net7.0-android</TargetFrameworks>
+    <TargetFrameworks>net8.0;net8.0-android</TargetFrameworks>
     <ImplicitUsings>enable</ImplicitUsings>
     <UseMaui>true</UseMaui>
   </PropertyGroup>
diff --git a/tests/WorkoutTracker.UnitTests/ViewModels/WorkoutViewModelTests.cs b/tests/WorkoutTracker.UnitTests/ViewModels/WorkoutViewModelTests.cs
new file mode 100644
index 0000000..13e39eb
--- /dev/null
+++ b/tests/WorkoutTracker.UnitTests/ViewModels/WorkoutViewModelTests.cs
@@ -0,0 +1,74 @@
+using FluentAssertions;
+using Microsoft.Extensions.Logging;
+using NSubstitute;
+using System.Collections.ObjectModel;
+using WorkoutTracker.MAUI.Interfaces;
+using WorkoutTracker.Models.Contracts;
+using WorkoutTracker.Models.Entities;
+using WorkoutTracker.Models.Presentation;
+using WorkoutTracker.Services;
+using WorkoutTracker.Services.Interfaces;
+using WorkoutTracker.Services.ViewModels;
+
+namespace WorkoutTracker.UnitTests.ViewModels;
+
+[TestFixture]
+public class WorkoutViewModelTests
+{
+    private WorkoutViewModel _vm;
+    private IWorkoutDataProvider _dataProvider;
+    
+    [SetUp]
+    public void Init()
+    {
+        _dataProvider = Substitute.For<IWorkoutDataProvider>();
+        var timer = Substitute.For<IExerciseTimerService>();
+        var context = new ApplicationContext<WorkoutViewModel>(Substitute.For<INotificationService>(),
+            Substitute.For<ILogger<WorkoutViewModel>>());
+        _vm = new WorkoutViewModel(_dataProvider, new SetsGenerator(_dataProvider), timer, context);
+    }
+
+    [Test]
+    public void AddingSetsShouldPersistTheRecord()
+    {
+        var model = new LogEntryViewModel
+        {
+            Id = Guid.NewGuid(),
+            Date = DateTime.UtcNow,
+            Sets = new ObservableCollection<IExerciseSet>(),
+            Order = 0,
+            Exercise = new ExerciseViewModel()
+        };
+        
+        _vm.AddSet(model);
+
+        model.Sets.Should().HaveCount(1);
+        model.Sets.Should().AllBeOfType<ProposedSet>();
+
+        _dataProvider.Received(1).UpdateViewModel(model);
+    }
+
+    [Test]
+    public void RemovingSetsShouldPersistTheRecord()
+    {
+        var initialSets = new ObservableCollection<IExerciseSet>();
+        initialSets.Add(new ProposedSet{Weight = 100, Repetitions = 10});
+        initialSets.Add(new ProposedSet{Weight = 150, Repetitions = 5});
+        initialSets.Add(new ProposedSet{Weight = 200, Repetitions = 2});
+        var model = new LogEntryViewModel
+        {
+            Id = Guid.NewGuid(),
+            Date = DateTime.UtcNow,
+            Sets = initialSets,
+            Order = 0,
+            Exercise = new ExerciseViewModel()
+        };
+        
+        _vm.ReduceSets(model);
+
+        model.Sets.Should().HaveCount(2);
+        model.Sets.Should().AllBeOfType<ProposedSet>();
+
+        _dataProvider.Received(1).UpdateViewModel(model);
+    }
+}
\ No newline at end of file
diff --git a/tests/WorkoutTracker.UnitTests/WorkoutTracker.UnitTests.csproj b/tests/WorkoutTracker.UnitTests/WorkoutTracker.UnitTests.csproj
index e0a23ad..335f67f 100644
--- a/tests/WorkoutTracker.UnitTests/WorkoutTracker.UnitTests.csproj
+++ b/tests/WorkoutTracker.UnitTests/WorkoutTracker.UnitTests.csproj
@@ -1,7 +1,7 @@
 <Project Sdk="Microsoft.NET.Sdk">
 
   <PropertyGroup>
-    <TargetFramework>net7.0</TargetFramework>
+    <TargetFramework>net8.0</TargetFramework>
     <ImplicitUsings>enable</ImplicitUsings>
     <Nullable>enable</Nullable>
     <UseMaui>true</UseMaui>