Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

양방향 연관관계 편의메서드 작성 #131

Merged
merged 12 commits into from
Nov 4, 2024
Merged

Conversation

kwonssshyeon
Copy link
Member

🔎 작업 내용

  • 게시글/댓글 연관관계 편의메서드 작성
  • 상위팀/하위팀 연관관계 편의메서드 작성
  • 게시글/좋아요 연관관계 편의메서드 작성

+ 테스트 파일 구조 수정 , 레포지토리 디렉토리 구분
+ not null 제약조건 추가

To Reviewers 📢

  • 연관관계 편의메서드 작성했는데 수정할 부분있는지 확인부탁드립니다 !
  • access model(post 편집권한) 계산하는 로직을 access model 클래스로 옮겼습니다. 이 클래스가 dto 디렉토리에 있는게 아닌거 같은데 어디로 옮길 수 있을까요?
  • 몇몇 도메인 로직은 엔티티 클래스로 옮겼습니다
  • 📢 아직 테스트코드 작성을 안해서 검증을 못해봤는데 일단 리뷰하고 테스트 코드는 이어서 작성하겠습니다.

체크 리스트

  • 테스트를 작성했습니다.
  • 테스트를 통과했습니다.
  • API 변경사항이 존재합니다.
  • API 호출을 직접 실시하였고, 해당 데이터가 정상적으로 표시됩니다.
  • 기존 코드에 영향을 주는 작업 내용이 존재합니다.
  • 향후 추가적인 작업이 필요한 부분이 있습니다.

➕ 관련 이슈

Copy link

github-actions bot commented Oct 31, 2024

Unit Test Results

16 tests   16 ✔️  1s ⏱️
  6 suites    0 💤
  6 files      0

Results for commit 7be901a.

♻️ This comment has been updated with latest results.

@chaejm55
Copy link
Contributor

chaejm55 commented Nov 1, 2024

다른 곳에서도 공통적으로 사용할 수 있으니 model 패키지를 새로 만들거나, 아니라면 원래대로 사용 클래스와 가장 가까운 dto/post에 두는 것이 맞다고 생각합니다.

@kwonssshyeon
Copy link
Member Author

다른 곳에서도 공통적으로 사용할 수 있으니 model 패키지를 새로 만들거나, 아니라면 원래대로 사용 클래스와 가장 가까운 dto/post에 두는 것이 맞다고 생각합니다.

아 dto는 데이터 다른 곳에서도 공통적으로 사용할 수 있으니 model 패키지를 새로 만들거나, 아니라면 원래대로 사용 클래스와 가장 가까운 dto/post에 두는 것이 맞다고 생각합니다.

dto는 순수 데이터 전달의 역할을 하니까 비지니스 로직이 포함된 access model을 어디에 위치시키는게 좋을지 여쭤본거였습니다 .!

@chaejm55
Copy link
Contributor

chaejm55 commented Nov 3, 2024

dto는 순수 데이터 전달의 역할을 하니까 비지니스 로직이 포함된 access model을 어디에 위치시키는게 좋을지 여쭤본거였습니다 .!

AccessModel 클래스에 대해 말씀드린게 맞습니다. 다른 곳에서 공통적으로 사용 가능하니 새로운 model 패키지 혹은 따로 하나만 빼기에 애매하면 살짝 맞지 않더라도 가장 가까운 dto 패키지에 위치 시키는게 맞다고 생각합니다.

@kwonssshyeon
Copy link
Member Author

dto는 순수 데이터 전달의 역할을 하니까 비지니스 로직이 포함된 access model을 어디에 위치시키는게 좋을지 여쭤본거였습니다 .!

AccessModel 클래스에 대해 말씀드린게 맞습니다. 다른 곳에서 공통적으로 사용 가능하니 새로운 model 패키지 혹은 따로 하나만 빼기에 애매하면 살짝 맞지 않더라도 가장 가까운 dto 패키지에 위치 시키는게 맞다고 생각합니다.

아 넵넵 ! 이렇게 사용하는게 하나뿐이라 그럼 dto 밑에 두겠습니다

@kwonssshyeon kwonssshyeon merged commit 2794e35 into dev Nov 4, 2024
3 checks passed
@kwonssshyeon kwonssshyeon deleted the Feat/issue-#130 branch November 4, 2024 07:54
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.

[Feature] 양방향 연관관계가 항상 유효하도록 편의메서드 작성
2 participants