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

Refactor#111 글 조회시 포지션과 역할 같이 반환하도록 수정 #111

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from

Conversation

jj0526
Copy link
Collaborator

@jj0526 jj0526 commented Jan 14, 2025

PR 내용

  • 글 조회시 파트랑 역할 같이 반환하도록 수정 하였습니다
  • 피그마에서 확인해보니, 공지사항, 게시판, 댓글에서 포지션이 필요해 추가하였습니다

PR 세부사항

  • 공지사항, 게시판, 댓글 조회시 포지션과 역할을 같이 반환하도록 수정하였습니다

관련 스크린샷

image


주의사항

없습니다

체크 리스트

  • 리뷰어 설정
  • Assignee 설정
  • Label 설정
  • 제목 양식 맞췄나요? (ex. #0 Feat: 기능 추가)
  • 변경 사항에 대한 테스트

@jj0526 jj0526 requested review from hyxklee and huncozyboy January 14, 2025 15:07
@jj0526 jj0526 self-assigned this Jan 14, 2025
@jj0526 jj0526 changed the title Refactor#105 글 조회시 포지션과 역할 같이 반환하도록 수정 Refactor#111 글 조회시 포지션과 역할 같이 반환하도록 수정 Jan 14, 2025
Comment on lines +52 to +53
Position position,
Role role,
Copy link
Member

Choose a reason for hiding this comment

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

고생하셨습니다 !
기존 DTO 형식에서 position, role만 추가된 내용이라 별다른 리뷰할 부분은 없었습니다.
저도 Mapper에서는 단순히 DTO와 Entity 간의 변환을 담당하는 역할만으로 제한해주는게 맞다고 생각하는 편입니다
그래서 비즈니스 로직을 Usecase에서 처리하도록 변경된 점은 좋다고 생각합니다

Copy link
Member

@hyxklee hyxklee left a comment

Choose a reason for hiding this comment

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

고생하셨어요!
리스트 조회랑 개별 조회 모두 알맞게 작업이 된 것 같습니당!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Refactor #105 게시글 목록 조회시 파트랑 역할 같이 보여주도록 수정
3 participants