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

위키 및 답변 삭제 기능을 구현한다. #177

Merged
merged 10 commits into from
Jan 24, 2025
Merged

위키 및 답변 삭제 기능을 구현한다. #177

merged 10 commits into from
Jan 24, 2025

Conversation

le2sky
Copy link
Member

@le2sky le2sky commented Jan 22, 2025

  • close 위키 및 답변 삭제 기능을 구현한다. #173
  • 위키와 답변 삭제 기능을 구현했어요. 논리 삭제이며, 답변이 존재하는 경우 위키를 삭제하지 못한다는 요구사항을 반영했습니다.
  • 코드 품질 관련해서 피드백 받고 싶은 부분이 있어서 코드 리뷰 요청 드립니다.
    1. comment와 wiki 패키지 간에 양방향 의존이 보여요. 개선할 필요가 있는지 의견이 궁금해요.
    2. 현재 정도 스케일의 프로덕트라면 크게 문제는 없을 지 모르지만, 논리 삭제라는 지식이 곳곳에 퍼져있어요. 예를 들어, CommentServicet에서 wiki를 찾아오기 위해서 wikiRepo.findByIdAndDeletedAt처럼 wiki가 논리 삭제인지에 대한 지식이 필요해요. 이 부분에 대한 의견이 궁금해요. 서비스끼리 협력하도록 구현해야하나 싶기도 하네요.

Copy link

github-actions bot commented Jan 22, 2025

Test Results

72 tests   72 ✅  3s ⏱️
28 suites   0 💤
28 files     0 ❌

Results for commit 737fd85.

♻️ This comment has been updated with latest results.

@le2sky le2sky requested a review from GIVEN53 January 22, 2025 05:38
Copy link
Member

@GIVEN53 GIVEN53 left a comment

Choose a reason for hiding this comment

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

comment와 wiki 패키지 간에 양방향 의존이 보여요. 개선할 필요가 있는지 의견이 궁금해요.

저도 조회를 구현하면서 고민하던 부분이었는데, 양방향 의존은 개선할 수 있을 때 빨리 하는 것이 좋다고 생각해요 미룰수록 더더욱 뜯기 힘들어질테니까요

현재 정도 스케일의 프로덕트라면 크게 문제는 없을 지 모르지만, 논리 삭제라는 지식이 곳곳에 퍼져있어요. 예를 들어, CommentServicet에서 wiki를 찾아오기 위해서 wikiRepo.findByIdAndDeletedAt처럼 wiki가 논리 삭제인지에 대한 지식이 필요해요. 이 부분에 대한 의견이 궁금해요. 서비스끼리 협력하도록 구현해야하나 싶기도 하네요.

논리 삭제 지식이 필요한 점이 저도 걸리긴 하네요 위에서 양방향 의존을 제거한다면 서비스간 협력도 괜찮지 않을까요??

@le2sky le2sky merged commit cbe7ecc into main Jan 24, 2025
3 checks passed
@le2sky le2sky deleted the 173 branch January 24, 2025 09:34
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.

위키 및 답변 삭제 기능을 구현한다.
2 participants