Merged
Conversation
Introduces the new CourtCaller.Persistence project with entity configurations, DbContext, and data seeding infrastructure. Refactors DAOs and tests to use the new persistence layer, removing the old DbContext from DAOs. Cleans up and standardizes method signatures and imports, and removes the WeatherForecast sample controller and model from the API.
Added using statements for CourtCaller.Persistence in multiple repository test files and included a project reference to CourtCaller.Persistence in UnitTests.csproj. This ensures the test projects have access to the necessary persistence layer for testing.
Introduces a robust database seeding system with configuration, dependency injection, and multiple seeding strategies. Adds migration for initial database creation, updates .gitignore, and modifies API startup to execute seeding in development. Also updates project files to include seed data and comments out direct role seeding in DbContext in favor of migration-based seeding.
Introduced seed data files for Bookings, Branches, Courts, News, Payments, Prices, RegistrationRequests, Reviews, TimeSlots, and UserDetails in the CourtCaller.Persistence/Data/Seed directory. These files provide sample data for application startup and testing, covering all major entities and their relationships as described in the included README.
Introduces a comprehensive ProductionDeploymentGuide.md detailing configuration, environment variables, seeding strategies, security, performance, monitoring, and deployment steps for running CourtCaller in Docker containers with production settings.
Refactored exception handling in JsonDataSeeder and FileReader to preserve original exception details and use more appropriate exception types. Updated BookingDAO to use FromSqlInterpolated for safer SQL queries. Modified PayLib to use ToLowerInvariant for IP address checks. Commented out appsettings.Development.json inclusion in CourtCaller.Persistence.csproj.
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
Introduced overloads for GetBranches and GetReview methods in DAOs, repositories, and services to support queries with and without status and search parameters. Updated method signatures to use nullable reference types and improved consistency across layers. Also changed some constants to static readonly for better flexibility.
Deleted redundant GetBranches methods that accepted only a status parameter from BranchDAO, BranchRepository, and BranchService. This streamlines the API and reduces maintenance of unused overloads.
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
Introduced a new BuildingBlocks.Logging project with Serilog integration, enrichers, and configuration options. Updated API to use the new logging setup and added the project reference. Refactored method signatures in DAOs, repositories, and services for consistency. Minor code formatting improvements and updated the solution file to include the new logging project.
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
1 participant
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.
No description provided.