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

[feat]Post #8

Closed
wants to merge 2 commits into from
Closed

[feat]Post #8

wants to merge 2 commits into from

Conversation

SWEETP0T4T0
Copy link
Collaborator

#️⃣이슈 번호

ex) close #이슈번호 와 같이 적어주세요.
close #7

📝작업 내용

이번 PR에서 작업한 내용을 자세히 설명해주세요. (이미지 첨부 가능)
기존 수업에서 authorEmail을 이용하여 author를 찾는 방식을 응용해서
userNickname이 Post 작성시에 기입되고 그를 이용해 user를 찾을 수 있게 구현하였음

스크린샷 (선택)

💬리뷰 요구사항(선택)

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

@SWEETP0T4T0 SWEETP0T4T0 marked this pull request as draft February 1, 2025 06:15
Copy link
Collaborator

@ki-hyun-Jang ki-hyun-Jang left a comment

Choose a reason for hiding this comment

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

  1. POST
    필요없는 부분
    title 제목
    appointment 예약 발행
    상기 2개 요구사항 정의서에 없음

그룹 (group_id)
share_post_id
post_visibility
post_del
score
hot_issue_yn
은 DB설계에 포함되어 있지만 현재 없음.

2.PostCreateReq
위와 같은이유로
PostCreateReq도 같이 수정되어야함

3.PostController
현재 팀에서 MVC방식이 아닌, RESTful API 방식으로 개발하기로 함.
retrun 타입이 전반적으로 ResponseEntity<?> 타입으로 수정되어야함.
Common패키지에서 팀에서 구성한 CommonDto, CommonErrorDto 참고
CommonExceptionHandler 로 컨트롤러에서 예외처리

해당 부분
https://awesome-currant-ce6.notion.site/REST-API-18cf0067972e80f69260c1532fb77e91?pvs=4
에 정리해두었으니 참고하세요.

@ki-hyun-Jang ki-hyun-Jang self-requested a review February 1, 2025 06:42
@ki-hyun-Jang ki-hyun-Jang marked this pull request as ready for review February 1, 2025 06:42
Copy link
Collaborator

@ki-hyun-Jang ki-hyun-Jang left a comment

Choose a reason for hiding this comment

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

상기 코멘트 확인하고 다시 PR 부탁드려요!

@leejaeseok-98
Copy link
Collaborator

  1. postController에 @GetMapping("/post")부분 "/post_list"으로 이름 변경.(post_detail도 함께)
  2. PostRepository에서 jpa 상속 해야함.

Copy link
Collaborator

@leejaeseok-98 leejaeseok-98 left a comment

Choose a reason for hiding this comment

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

admin에서 다른 엔티티들을 가져와야 하는데 공통 엔티티가 필요할거 같아서 내일 회의 시간에 이야기 해보고 각 도메인들을 개별패키지에 둘지 아님 공통엔티티에 둘지 정해봐야할 것 같습니다!

@SWEETP0T4T0 SWEETP0T4T0 closed this Feb 6, 2025
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.

[feat]Post 초기
3 participants