Skip to content

Feature/rating service#61

Merged
muhammadzkralla merged 11 commits into
developfrom
feature/rating-service
Jun 27, 2026
Merged

Feature/rating service#61
muhammadzkralla merged 11 commits into
developfrom
feature/rating-service

Conversation

@NaderMohamed325

Copy link
Copy Markdown
Member

Summary

This pull request refactors and enhances the Rating management features. It introduces unified DTOs for rating responses, adds create/update DTOs with validation, implements a dedicated RatingController, and sets up the Rating entity and service for better handling of user feedback and trip ratings.

Changes Summary

  • Added RatingResponse DTO for consistent API responses.
  • Added DTOs for rating creation and update workflows, including input validation.
  • Created RatingController to facilitate operations on trip ratings.
  • Built Rating entity and service for feedback and ratings management.

@muhammadzkralla
muhammadzkralla requested review from Copilot and muhammadzkralla and removed request for Copilot June 26, 2026 13:30

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.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

Comment thread src/main/java/com/yaquodorg/yaquod/controller/RatingController.java Outdated
Comment thread src/main/java/com/yaquodorg/yaquod/service/rating/RatingServiceImpl.java Outdated
Comment thread src/main/java/com/yaquodorg/yaquod/service/rating/RatingServiceImpl.java Outdated
Comment thread src/main/java/com/yaquodorg/yaquod/repository/RatingRepository.java
Comment thread src/main/java/com/yaquodorg/yaquod/repository/RatingRepository.java
Comment thread src/main/java/com/yaquodorg/yaquod/service/rating/RatingServiceImpl.java Outdated
muhammadzkralla and others added 5 commits June 26, 2026 17:45
…ById() method bad transactional handling and replaced it with raw delete query.
…ture/rating-service

# Conflicts:
#	src/main/java/com/yaquodorg/yaquod/service/rating/RatingServiceImpl.java

@muhammadzkralla muhammadzkralla left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

All good, thank you <3
عاش يقود

@muhammadzkralla
muhammadzkralla merged commit aab7c50 into develop Jun 27, 2026
1 check passed
@muhammadzkralla muhammadzkralla added the enhancement New feature or request label Jun 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants