Skip to content

Sprint 17 Stage 3 Comments solution - #14

Merged
Naz1anmak merged 2 commits into
mainfrom
feature/comments
Oct 24, 2025
Merged

Sprint 17 Stage 3 Comments solution#14
Naz1anmak merged 2 commits into
mainfrom
feature/comments

Conversation

@internet-prodashi

@internet-prodashi internet-prodashi commented Oct 22, 2025

Copy link
Copy Markdown
Collaborator

Завершен 3-й этап спринта - реализована фича "комментарии к событию"

@Naz1anmak Naz1anmak changed the title The 3rd stage of the final task Sprint 17 Stage 3 Comments solution Oct 22, 2025
@Naz1anmak
Naz1anmak requested a review from Copilot October 22, 2025 09:17

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Реализована функциональность комментариев к событиям (Event comments) в рамках третьего этапа финального задания. Добавлены три уровня доступа: административный (удаление любых комментариев), приватный (CRUD операции пользователя над собственными комментариями) и публичный (просмотр комментариев события).

Основные изменения:

  • Создана полная инфраструктура для работы с комментариями: модель, репозиторий, сервис, контроллеры и DTO
  • Добавлена таблица БД с индексами для оптимизации выборок
  • Реализованы REST API эндпоинты для всех уровней доступа
  • Подготовлена коллекция Postman тестов для проверки функциональности

Reviewed Changes

Copilot reviewed 14 out of 14 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
postman/feature.json Коллекция Postman тестов для проверки всех операций с комментариями
main-service/src/main/resources/schema.sql Определение таблицы comments с индексами по event_id и user_id
main-service/src/main/java/ru/practicum/comment/service/CommentServiceImpl.java Реализация бизнес-логики для создания, обновления, удаления и получения комментариев
main-service/src/main/java/ru/practicum/comment/service/CommentService.java Интерфейс сервиса комментариев
main-service/src/main/java/ru/practicum/comment/repository/CommentRepository.java Репозиторий с кастомными запросами для выборки комментариев с использованием JOIN FETCH
main-service/src/main/java/ru/practicum/comment/model/Comment.java JPA-сущность для хранения комментариев
main-service/src/main/java/ru/practicum/comment/mapper/CommentMapper.java MapStruct маппер для преобразования между entity и DTO
main-service/src/main/java/ru/practicum/comment/dto/*.java DTO классы для создания, обновления и представления комментариев
main-service/src/main/java/ru/practicum/comment/controller/*.java REST контроллеры для административного, приватного и публичного доступа
README.md Техническая документация API комментариев

Comment thread main-service/src/main/java/ru/practicum/comment/service/CommentServiceImpl.java Outdated
Comment thread main-service/src/main/java/ru/practicum/comment/service/CommentServiceImpl.java Outdated
Comment thread postman/feature.json Outdated
Comment thread README.md
Comment thread main-service/src/main/java/ru/practicum/comment/service/CommentServiceImpl.java Outdated
Comment thread main-service/src/main/java/ru/practicum/comment/service/CommentServiceImpl.java Outdated
Comment thread main-service/src/main/java/ru/practicum/comment/service/CommentServiceImpl.java Outdated
Comment thread main-service/src/main/java/ru/practicum/comment/service/CommentServiceImpl.java Outdated
@Naz1anmak Naz1anmak self-assigned this Oct 22, 2025
@Naz1anmak
Naz1anmak self-requested a review October 22, 2025 13:22
Comment thread main-service/src/main/java/ru/practicum/comment/repository/CommentRepository.java Outdated
@Naz1anmak
Naz1anmak requested a review from Copilot October 23, 2025 21:44

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Copilot reviewed 31 out of 31 changed files in this pull request and generated 1 comment.

Comments suppressed due to low confidence (1)

main-service/src/main/java/ru/practicum/category/service/CategoryServiceImpl.java:75

  • Переменная categoriesPage имеет неправильное название, так как теперь она содержит List, а не Page. Следует переименовать в categories.
        List<Category> categoriesPage = categoryRepository.findAllList(pageable);

        if (categoriesPage.isEmpty()) {
            return List.of();
        }

Comment thread main-service/src/main/java/ru/practicum/comment/dto/CommentDto.java Outdated
…ve data retrieval methods; update DTOs for consistency
@Naz1anmak
Naz1anmak requested review from Naz1anmak and afomkina and removed request for Naz1anmak and afomkina October 23, 2025 21:53

@afomkina afomkina left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Получилась отличная фича! Молодец

@Naz1anmak
Naz1anmak merged commit 802fae2 into main Oct 24, 2025
2 checks passed
@Naz1anmak
Naz1anmak deleted the feature/comments branch October 24, 2025 18:58
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.

4 participants