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

Provide DTO for transfer between API and Application layers #146

Conversation

StraykerPL
Copy link
Member

@StraykerPL StraykerPL commented Jan 9, 2024

Summary

Make API endpoints work only on DTOs and Application layer to work only on Core models.

Control List

  • Code formatting checked,
  • Coding Guidelines compliance checked,
  • Documentation updated (if necessary),
  • Automated tests updated (if necessary),

Screenshots

Work under this PR is not visual, no screenshots available.

@StraykerPL StraykerPL added the feature New feature or content label Jan 9, 2024
@StraykerPL StraykerPL self-assigned this Jan 9, 2024
@StraykerPL StraykerPL linked an issue Jan 9, 2024 that may be closed by this pull request
Lukash88
Lukash88 previously approved these changes Jan 20, 2024
…een-api-and-application-layers

# Conflicts:
#	binder-web-backend/Binder.Api/Controllers/ToDoTasksController.cs
#	binder-web-backend/Binder.Application/Services/Middleware/ErrorHandlingMiddleware.cs
#	binder-web-frontend/src/api/.openapi-generator/FILES
#	binder-web-frontend/src/api/api/tables.service.ts
#	binder-web-frontend/src/api/api/toDoTasks.service.ts
#	binder-web-frontend/src/api/model/defaultTableDTO.ts
#	binder-web-frontend/src/api/model/models.ts
#	binder-web-frontend/src/api/model/toDoTaskDTO.ts
#	binder-web-frontend/src/pages/home/components/table-view/table-view.component.ts
#	binder-web-frontend/src/shared/services/activeTable.service.ts
@StraykerPL StraykerPL merged commit c80e6ee into develop Jan 30, 2024
2 checks passed
@StraykerPL StraykerPL deleted the feature/113-provide-dto-for-transfer-between-api-and-application-layers branch January 30, 2024 22:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or content
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Feature: Provide DTO for transfer between API and Application layers
3 participants