이미지 관리 테이블 작성한다

[jjt4515]

해결하려는 문제가 무엇인가요?

• close 이미지 관리 Table 작성한다 #33

어떻게 해결했나요?

• 이미지 정보 저장 CRUD 구현
• presigned Url 을 통해 s3에 이미지를 업로드 한 뒤 클라이언트에서 또 요청을 보내 db에도 저장을 해야 해요

코드 리뷰시 요청 사항

• 더 필요한 부분이 있을까요?

더 하고 싶은 말

• 이슈 사항
  1. jpg 말고 다른 형식 파일 들어오는 경우에 대해서도 따로 처리 필요
  2. 주기적으로 s3 정리하는 스케줄러 구현 필요(사진 5개 이상이면 db에 저장 안 한 것들은 지움)
  3. 극악의 확률로 url이 겹치면?? -> 그럴 일 거의 없긴할텐데 생기면 s3 원래 파일 지워집니다. 우선은 url 변수 더 추가했습니다.

하다 보니 위의 이슈 사항이 생겼는데, 나중에 리팩토링 해보도록 할게요.!
일단은 api 명세도 빠른 시일 내에 해보겠습니다.
week7이 없어서 일단 week6으로 pr 올렸는데, week7 그냥 만들면 되나요?

[amm0124]

week7 만들었습니다 ~

[canyos]

저는 review만들면서 referenceId 대신에 entityId 변수 명을 사용했는데 뭐가 더 직관적인거 같나여?

[jjt4515]

오 둘 다 괜찮아서 못 고르겠네요

저는 review만들면서 referenceId 대신에 entityId 변수 명을 사용했는데 뭐가 더 직관적인거 같나여?
오 둘 다 괜찮은데요?
둘 다 상관없을 것 같습니다!

[stopmin]

수고하셨습니다!

잘 동작하면 괜찮을 것 같습니다!!

[stopmin]

여기 /빠진 것 같습니당~~

[jjt4515]

삭제된 이미지와 똑같은 이미지를 사용자가 다시 올릴 경우, 삭제된 이미지의 deleted_at을 null로 바꿔서 다시 복구가 되는 것처럼 구현하였는데 괜찮을까요?
회원 계정 같은 게 아니고 이미지라 문제 없을 거 같기도 하고 똑같은 거 삭제했다가 다시 저장할 때마다 새로 만들면 db에 자리를 많이 차지할 것 같다는 생각이 들었습니다.

따로 복구 로직도 만들긴 했습니다!
그런데 이미지는 복구할 일이 없을 것 같긴 해서, 하드 삭제를 해야 하나 생각이 또 드네요