[#21] 작품 등록 #27
[#21] 작품 등록 #27
Conversation
f-lab-saponin
left a comment
f-lab-saponin
left a comment
There was a problem hiding this comment.
고생하셨습니다~!
사용자, 인증, 작가에 대한 부분도 여기 PR 에 같이 있는것같아요. 다음부터는 PR 제목과 연관된 코드만 있도록 한번 부탁드립니다!
리뷰드린 부분 확인 부탁드립니다 감사합니다
| FROM public.ecr.aws/lambda/nodejs:20 | ||
|
|
||
| # sharp + AWS SDK 설치 | ||
| RUN npm install --omit=dev sharp @aws-sdk/client-s3 | ||
|
|
||
| # 코드 복사 | ||
| COPY index.js ${LAMBDA_TASK_ROOT}/index.js | ||
|
|
||
| # 핸들러 지정 (index.js의 exports.handler) | ||
| CMD [ "index.handler" ] No newline at end of file |
There was a problem hiding this comment.
이 이미지를 만들어서 어디에 올렸는지도 관리를 위해 md 파일에 한번 적어주세요. 🙂
src/main/java/com/creatorhub/constant/CreationThumbnailType.java
Outdated
Show resolved
Hide resolved
src/main/java/com/creatorhub/controller/S3PresignedUploadController.java
Outdated
Show resolved
Hide resolved
| @ManyToOne(fetch = FetchType.LAZY) | ||
| @JoinColumn(name = "creation_id", nullable = false) | ||
| private Creation creation; |
There was a problem hiding this comment.
양방향 관계가 필요한지 확인해보시고, 필요없으면 설계를 단순화하기 위해 단방향으로 하면 좋아보여요.
There was a problem hiding this comment.
Creation-CreationThumbnail, Creation-CreationHashtag 은 양방향, CreationHashtag-Hashtag는 단방향으로 최종적으로 fix했습니다.
CreationThumbnail 과 CreationHashtag 은 Creation에 주도권을 줘서 CreationThumbnail, CreationHashtag 의 변경/삭제/정렬 규칙을 작성하는게 좋을 거 같아 우선 양방향으로 설정했습니다.
작품 등록 시 해시태그는 자동 완성 기능으로 선택되며 이미 DB에 존재하는 참조 데이터 이므로
CreationHashtag–Hashtag는 양방향이 아닌 단방향 연관관계로 설계했습니다.
|
네 알겠습니다! 리드미 파일이 아직 정리가 안됐는데 말씀하신 부분들 취합해서 차후 정리하겠습니다. 감사합니다~ |
작업 내용
테스트 내용
추가 필요