Skip to content

Conversation

@bbbbooo
Copy link
Member

@bbbbooo bbbbooo commented Dec 23, 2024

#️⃣연관된 이슈

ex) #이슈번호, #이슈번호

📝작업 내용

이번 PR에서 작업한 내용을 간략히 설명해주세요(이미지 첨부 가능)

  • 인증 및 노트 상세 조회 구현했습니다.
  • 이전 엔티티 추가하는 PR에서 언급하신 연관관계 설정도 완료했습니다.

스크린샷 (선택)

  • 댓글 없을때

image

  • 댓글 있을때

image

💬리뷰 요구사항(선택)

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

ex) 메소드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?

@soheeparklee
Copy link
Contributor

soheeparklee commented Dec 23, 2024

댓글 응답에 유저 이름도 줘야하지 않을까요? 피그마 디자인 보니까 이런 방향으로 갈 것 같아서요.
링크, status, start date는 줘야할지 안줘도 될지 모르겠네요..

Screenshot 2024-12-23 at 16 08 12

@bbbbooo
Copy link
Member Author

bbbbooo commented Dec 23, 2024

댓글 응답에 유저 이름도 줘야하지 않을까요? 피그마 디자인 보니까 이런 방향으로 갈 것 같아서요. 링크, status, start date는 줘야할지 안줘도 될지 모르겠네요..

Screenshot 2024-12-23 at 16 08 12

아 맞네요 이름 추가해두겠습니다. 나머지는 일단 응답에 넣어두었습니다.

@soheeparklee soheeparklee merged commit b10c30a into develop Dec 23, 2024
1 check passed
@soheeparklee soheeparklee deleted the feature/complete-detail branch December 23, 2024 16:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

인증 및 노트 상세 조회

3 participants