Skip to content

Grzegorz Jaczewski - Refactoring of BrainWare #24

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 11 commits into
base: main
Choose a base branch
from
Open

Grzegorz Jaczewski - Refactoring of BrainWare #24

wants to merge 11 commits into from

Conversation

grzesiekjaczewski
Copy link

Changes in BrainWare project:

  • Refactored to use dependency injection; classes are now registered in builder.Services.
  • Replaced direct database access with Entity Framework.
  • Separated model classes into distinct files.
  • Moved the connection string to application.json.
  • Extracted database querying from OrderService.
  • Replaced the .NET Framework unit test project with a .NET-based project.
  • Added tests for OrderService.
  • Modified the controller endpoint to return ActionResult.

Further possible enhancements:

  • UI for displaying the order list.
  • Authorization and user management.
  • Unit tests for the endpoints.
  • Integration tests for the database.

@grzesiekjaczewski
Copy link
Author

I have pushed another commit that contains:

  • The refactored Angular project to connect with the API project port
  • Extracted API queries into a separate service
  • Upgraded to version 17
  • Added the Angular project to the solution
  • Added CORS settings to program.cs
  • Some minor refactoring of the API project

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@grzesiekjaczewski