Skip to content

feat: 보드 유저 확인#32

Merged
89JHoon merged 2 commits intodevelopfrom
feature/board
Jan 14, 2025
Merged

feat: 보드 유저 확인#32
89JHoon merged 2 commits intodevelopfrom
feature/board

Conversation

@sumyeom
Copy link
Contributor

@sumyeom sumyeom commented Jan 13, 2025

  1. 보드 생성 시에 유저 추가
  2. 보드 수정, 삭제 시에 로그인한 유저 확인 추가
  3. 보드 수정, 삭제 시에 return 값 Void로 변경

#️⃣연관된 이슈

#28

📝작업 내용

보드 기능에서 유저 관련하여 필요한 부분 추가

스크린샷 (선택)

💬리뷰 요구사항(선택)

리뷰 부탁드립니다.

1. 보드 생성 시에 유저 추가
2. 보드 수정, 삭제 시에 로그인한 유저 확인 추가
3. 보드 수정, 삭제 시에 return 값 Void로 변경

// 게시글 작성자와 로그인한 사용자 확인
if(!findBoard.getUser().getId().equals(loginUser.getId())) {
throw new ApiException(FORBIDDEN);
Copy link
Contributor

Choose a reason for hiding this comment

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

저희 에러코드 정적 임포트 해서 쓰나요?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

수정해서 올리겠습니다

Copy link
Contributor

@Newbiekk-kkh Newbiekk-kkh left a comment

Choose a reason for hiding this comment

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

확인했습니다.

1. ErrorCode.형식으로 수정
@sumyeom sumyeom self-assigned this Jan 13, 2025
@89JHoon 89JHoon merged commit 3bf77fe into develop Jan 14, 2025
yeji-world pushed a commit that referenced this pull request Jan 14, 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.

4 participants