refactor no-personal-data-in-public-handles-and-queries-to-cqrs#227
Open
NikolaJJMusatov wants to merge 1 commit intodevelopmentfrom
Open
refactor no-personal-data-in-public-handles-and-queries-to-cqrs#227NikolaJJMusatov wants to merge 1 commit intodevelopmentfrom
NikolaJJMusatov wants to merge 1 commit intodevelopmentfrom
Conversation
Contributor
NikolaJJMusatov
commented
Nov 28, 2024
- Интерфейсы PublicTaskResponseDto и PublicTasksResponseDto определены в src/common/types/api.types.ts.
- Метод TasksService.getAllVirginTasks() изменён так, что ReturnType<TasksService.getAllVirginTasks()> === Array
- Папка src/common/commands переименована в src/common/commands-and-queries, проверена корректная работа приложения после переименования
- Создан запрос получения публичных заявок.
- Создан обработчик запроса публичных заявок.
- В SystemApiController заменено прямое обращение к сервису заявок на вызов соответствующего запроса через QueryBus.
- Создан запрос получения новых задач для авторизованного пользователя.
- Создан обработчик запроса новых задач для авторизованного пользователя.
- QueryBus импортирован из пакета '@nestjs/cqrs' и инжектирован в конструктор VolunteerApiController.
- В VolunteerApiController.getNewTasks() заменено прямое обращение к сервису заявок на вызов соответствующего запроса через QueryBus.
- QueryBus импортирован из пакета '@nestjs/cqrs' и инжектирован в конструктор RecipientApiContoller.
- В RecipientApiContoller.getVirginTasks() заменено прямое обращение к сервису заявок на вызов соответствующего запроса через QueryBus.
- Работоспособность ручки GET /system/tasks/virgin проверена через Postman/Insomnia.
- Работоспособность ручки GET /volunteer/tasks/virgin проверена через Postman/Insomnia.
- Работоспособность ручки GET /recipient/tasks/virgin проверена через Postman/Insomnia.
15 tasks
kspshnik
approved these changes
Nov 28, 2024
Collaborator
kspshnik
left a comment
There was a problem hiding this comment.
Супер, отличная работа! Спасибо!
@INextYP, ваше слово, товарищ маузер твоя очередь :)
Количества файлов не пугайся, там 80% - переименование src/common/commands -> src/common/commands-and-queries, как самих файлов, так и в путях импорта :)
INextYP
approved these changes
Nov 30, 2024
Collaborator
INextYP
left a comment
There was a problem hiding this comment.
Всё хорошо, отличная работа!
22 tasks
22 tasks
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
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.