Refactor #200 게시판 작성 request 변경 및 게시판 수정 api 분리 #200
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
14 tasks
hyxklee
approved these changes
Aug 12, 2025
Collaborator
hyxklee
left a comment
hyxklee
left a comment
There was a problem hiding this comment.
고생하셨습니다!
PATCH에 맞게 null check가 잘 이루어지고 있는 것 같아요!
게시판 관련 사항은 프론트 담당자 분이랑 잘 소통하면서 해주시고 있는 것 같아서 좋습니다 👍🏻
jj0526
approved these changes
Aug 12, 2025
|
|
||
| public void updateEducation(PostDTO.UpdateEducation dto) { | ||
| this.updateUpperClass(dto); | ||
| this.part = null; |
Collaborator
There was a problem hiding this comment.
원래 교육용 자료에서는 part가 쓰이지 않는 컬럼인데 null로 업데이트한 이유가 있을까요?
Member
Author
There was a problem hiding this comment.
교육자료에서는 part 미사용이라는 도메인 규칙을 재확인하려는 의도로 설계했습니다 !
정상적인 플로우는 아니지만 교육자료를 여러번 업데이트할때 옛 값이 남을 수 있는 경우를 방지하고자
업데이트시에도 null값으로 명시적으로 지정해줬다고 생각해주심 될거같아요
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
PR 내용
게시판 디자인이 변경됨에 따라서, 새로 바뀐 뷰에 맞춰서 아래 내용들을 수정했습니다
PR 세부사항
파트 게시판 작성
request변경교육자료 게시판 작성
request변경게시판 수정 API 분리
파트 게시판 수정 API
제목 + 내용 + 파일에 추가 필드들 반영 (스터디 이름, 주차, 기수, 파트)
제목 + 내용 + 파일 (기수, 파트 리스트)
게시판 상세조회 응답 dto 수정
관련 스크린샷
주의사항
게시글 수정방식이 명시적으로 모든 필드들 보내주기보다는, 수정이 필요한 부분만 입력받는 로직으로 변경되었으니 꼼꼼하게 봐주시면 감사하겠습니다 🙇🏻♂️
체크 리스트