backend call for dates implemented#650
Merged
StuartFerguson merged 6 commits intomainfrom Jan 15, 2026
Merged
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
This pull request implements backend API calls for fetching comparison dates from an estate reporting service. The PR introduces a new BusinessLogic layer that separates concerns and implements request handlers using MediatR pattern to make actual API calls instead of returning mock data.
Changes:
- Introduced new
EstateManagementUI.BusinessLogicproject with API client, request handlers, and data transfer objects - Implemented
DateRequestHandlerto fetch comparison dates from the backend API - Refactored code to use
SimpleResultsResult type instead of custom Result wrapper - Added model conversion layer (
ModelFactory) to bridge BusinessLogic and BlazorServer models
Reviewed changes
Copilot reviewed 53 out of 53 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| EstateManagmentUI.BusinessLogic/RequestHandlers/DateRequestHandler.cs | New request handlers for comparison dates and other entities |
| EstateManagmentUI.BusinessLogic/Client/DateMethods.cs | API client method for fetching comparison dates |
| EstateManagmentUI.BusinessLogic/BackendAPI/IEstateReportingApiClient.cs | Backend API client interface and implementation |
| EstateManagmentUI.BusinessLogic/EstateManagementUI.BusinessLogic.csproj | New business logic project configuration |
| EstateManagementUI.BlazorServer/Program.cs | Updated DI configuration to use new request handlers |
| EstateManagementUI.BlazorServer/Components/Pages/Home.razor.cs | Refactored to use new model factory and authentication state |
| EstateManagementUI.BlazorServer/Factories/ModelFactory.cs | New factory for converting between BusinessLogic and BlazorServer models |
| Multiple Razor pages | Updated to use ModelFactory for data conversion |
| Test files | Updated mocks to use SimpleResults Result type |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
closes #601