Skip to content

Refactor mappers and enhance view models#18

Draft
jdubar wants to merge 12 commits intomainfrom
handle-deleted-bowlers
Draft

Refactor mappers and enhance view models#18
jdubar wants to merge 12 commits intomainfrom
handle-deleted-bowlers

Conversation

@jdubar
Copy link
Copy Markdown
Owner

@jdubar jdubar commented Oct 28, 2025

Refactor mappers and enhance view models

Refactor code to replace mapper classes with extension methods, simplifying mapping between models and view models. Remove IMapper interface and its implementations, moving logic to static methods in BowlerMappers, PersonMappers, and WeekMappers. Update view models to use object initializers and property initializers, removing constructors. Enhance Stepper control with command support. Update CurrentWeekOverviewPage.xaml to bind Stepper commands. Modify MauiProgram to remove mapper service registrations. Update Person class with IsDeleted property. Adjust package references in HangTab.csproj.

Refactor code to replace mapper classes with extension methods, simplifying mapping between models and view models. Remove `IMapper` interface and its implementations, moving logic to static methods in `BowlerMappers`, `PersonMappers`, and `WeekMappers`. Update view models to use object initializers and property initializers, removing constructors. Enhance `Stepper` control with command support. Update `CurrentWeekOverviewPage.xaml` to bind `Stepper` commands. Modify `MauiProgram` to remove mapper service registrations. Update `Person` class with `IsDeleted` property. Adjust package references in `HangTab.csproj`.
@jdubar jdubar self-assigned this Oct 28, 2025
@jdubar jdubar added enhancement New feature or request refactor Improve code labels Oct 28, 2025
@codecov
Copy link
Copy Markdown

codecov bot commented Oct 28, 2025

Codecov Report

❌ Patch coverage is 93.96552% with 7 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
HangTab/Mappers/BowlerMappers.cs 92.68% 0 Missing and 3 partials ⚠️
HangTab/Mappers/PersonMappers.cs 91.17% 0 Missing and 3 partials ⚠️
HangTab/Mappers/WeekMappers.cs 93.75% 0 Missing and 1 partial ⚠️
Files with missing lines Coverage Δ
HangTab/Mappers/ViewModelMappers.cs 100.00% <100.00%> (ø)
HangTab/Models/Person.cs 100.00% <100.00%> (ø)
HangTab/Mappers/WeekMappers.cs 93.75% <93.75%> (ø)
HangTab/Mappers/BowlerMappers.cs 92.68% <92.68%> (ø)
HangTab/Mappers/PersonMappers.cs 91.17% <91.17%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request refactor Improve code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant