COMMENTS-PRIVATE: Создание/обновление комментария пользователем #84 и #86

[Gagarskiy-Andrey]

Реализован эндпоинт POST /users/{userId}/comments?eventId={eventId} и соответствующая логика сервиса для создания нового комментария авторизованным пользователем.

Реализован эндпоинт PATCH /users/{userId}/comments/{commentId} и соответствующая логика сервиса для обновления пользователем текста своего комментария. Обновление возможно только в течение 6 часов после создания и если комментарий не был "мягко" удален.

Прописаны:

• Service (CommentService интерфейс и CommentServiceImpl);
• Controller (PrivateCommentController);
• Юнит-тесты для сервиса и контроллера.

[SergikF]

Критичных замечаний у меня нет, свои мысли представил в необязательных комментах.

[SergikF]

[НЕОБЯЗАТЕЛЬНО] Хоть я и склоняюсь к мысли, что требование передавать в методах не более 2-х параметров - это определенное требование, но, по моему, в коде были такие методы и это не вызвало замечаний ревьювера.

[Gagarskiy-Andrey]

Я понимаю о чем ты, об этом говорил старший преподаватель, однако в тз такого требования нет. И видимо, ревьюеру это не принципиально, т.к. параметров не на столько много, чтобы их количество вызывало трудности в разборе написанного кода.

[impatient0]

Хорошая реализация! Логика отлично прописана; есть только пара моментов, которые стоит поправить, прежде чем можно будет мёрджить – написал в комментариях.

[progingir]

работа супер, оставила пару комментов, в целом, ребята там за меня уже достаточно написали :)

[progingir]

[НЕОБЯЗАТЕЛЬНО] в тестах можно добавить проверку на корректность значений полей createdOn, updatedOn, isEdited в возвращаемом CommentDto. сейчас ты проверяешь только текст и флаг isEdited

[progingir]

крутышка

[impatient0]

Всё супер, мёрджим!