Skip to content

Conversation

@DachiSkhiladze
Copy link

@DachiSkhiladze DachiSkhiladze commented May 23, 2023

This pull request incorporates the requested changes for Task #1 in the VelvetechTestTask project. The following modifications have been implemented:

Swagger Integration: Added Swagger documentation to the service, offering an interactive API interface for easier exploration and testing of endpoints.

Database Migration: Replaced the previous InMemoryDB implementation with SQL Server to persist the task list. The connection string is now stored in the appsettings configuration file, facilitating seamless configuration and deployment.

Error Logging with Serilog: Implemented error logging using Serilog, a versatile logging framework. This allows for efficient tracking and analysis of errors, enhancing troubleshooting capabilities.

Refactoring: Introduced an Infrastructure layer to serve as the Data Access Layer, responsible for handling database interactions. Additionally, an Application layer was introduced to encapsulate the business logic and orchestrate data operations.

I considered utilizing the CQRS (Command Query Responsibility Segregation) pattern initially. However, upon careful consideration, I deemed it as overkill for such a small-scale application. Therefore, I decided to opt for a simpler approach.

Furthermore, I upgraded the project to utilize .NET 6.0, leveraging its latest features and improvements.

Please review the changes made in this pull request and I will appreciate feedback.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant