Skip to content

Conversation

@Nohheechul
Copy link
Collaborator

📌 PR 제목

데이터 파일 수정, 엔티티 수정, dto 수정

@Nohheechul Nohheechul requested a review from Copilot September 26, 2025 08:42
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates entity mappings and documentation examples across the lawyer application domain model. The changes primarily focus on adding explicit table name mappings to JPA entities and updating API documentation examples to use more relevant legal terminology.

  • Adds explicit @Table annotations to all law-related entities for clear database table mapping
  • Updates API documentation examples from labor law ("노동") to criminal law ("절도", "형사") terminology
  • Enhances Docker configuration with SQL file mounting for easier database management

Reviewed Changes

Copilot reviewed 10 out of 13 changed files in this pull request and generated no comments.

Show a summary per file
File Description
Precedent.java Added explicit table name mapping
PrecedentSearchRequestDto.java Updated example from "노동" to "절도"
LawWord.java Added explicit table name mapping
Law.java Added explicit table name mapping
Jo.java Added explicit table name mapping
Jang.java Added explicit table name mapping
Ho.java Added explicit table name mapping
Hang.java Added explicit table name mapping
LawSearchRequestDto.java Updated examples to criminal law terminology
docker-compose.yml Added SQL folder mounting for database setup

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Copy link
Collaborator

@DooHyoJeong DooHyoJeong left a comment

Choose a reason for hiding this comment

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

good

@Nohheechul Nohheechul merged commit 1c71734 into develop Sep 26, 2025
4 checks passed
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.

3 participants