Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Dgdimitrov/refactoring #41

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

ddidgdimitrov-infragistics
Copy link
Collaborator

-Introduce BaseDto and BaseDb to centralize common properties and functionality.
-Refactor with BaseNorthwindAPIController to eliminate redundant code across controllers.
-Implement BaseDbService to streamline business logic and reduce repetition in controllers.
-Remove direct database interactions from controllers, delegating all logic to services.
-Remove database objects from tests, ensuring all logic is managed through services for better testability.

When AB is ready, we will update all DTOs to have internal setters for IDs and pass the ID via query parameters for the PUT update method. The Update method in BaseDbService is already prepared for this change. I will submit a PR for this soon.

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.

2 participants