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

✨ [STMT-168] 스터디 삭제 api 및 s3에 저장된 관련 파일 삭제 기능 구현 #118

Merged
merged 13 commits into from
Apr 25, 2024

Conversation

05AM
Copy link
Collaborator

@05AM 05AM commented Apr 22, 2024

💁 해결 하려는 문제를 적어주세요

  • 스터디 삭제 기능을 구현합니다.

🤔 어떤 방식으로 해결했는지 적어주세요

  • 스터디 삭제 api
  • 전에 논의했던 내용대로 스터디는 soft delete하고, study member는 다 삭제, study tag는 남겨두었습니다.
  • 혹시 이에 대해서 추가 의견이 있다면 코멘트 부탁 드립니다.
  • 관련 s3 객체 일괄 삭제
  • 현재 study/{studyId}/main과 study/{studyId}/activity 경로로 파일 객체들이 저장되고 있는데, studyId를 이용하여 해당하는 폳더를 삭제하는 기능을 구현했습니다.

🙋 중점적으로 리뷰 했으면 하는 부분이 있다면 적어주세요

  • soft delete에 대한 의견이 궁금합니다.
  • 어떤 것을 남기고, 어떤 것을 삭제할지에 대한 의견이 추가로 있으시면 공유해주세요!
  • 최대 첨부파일 크기를 클라이언트에서 압축을 해온다는 가정하에 5mb로 정했는데 이에 대한 의견이 궁금합니다.
  • test setup 파일에서 member 1을 admin, member 2를 그냥 study member, member 3을 외부자로 추가하면서 테스트 관련 코드가 조금 변경됐는데 이에 대해 의견이 있으시다면 공유해주시면 감사하겠습니다!

@05AM 05AM added the D-2 2일 전 까지 리뷰해주세요 label Apr 22, 2024
@05AM 05AM self-assigned this Apr 22, 2024
@05AM 05AM requested a review from zxcv9203 as a code owner April 22, 2024 06:46
Copy link
Collaborator

@zxcv9203 zxcv9203 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다 💪

@zxcv9203
Copy link
Collaborator

  • soft delete에 대한 의견이 궁금합니다.

👍👍👍👍👍👍👍👍👍👍👍👍👍👍👍👍👍👍👍👍👍👍👍👍👍👍👍👍👍👍👍 💪💪💪💪💪💪💪💪💪💪💪💪💪💪💪💪💪

  • 어떤 것을 남기고, 어떤 것을 삭제할지에 대한 의견이 추가로 있으시면 공유해주세요!

지금같이 유지하고 추가로 내용이 변경될때 대응하면 될 듯 합니다

  • 최대 첨부파일 크기를 클라이언트에서 압축을 해온다는 가정하에 5mb로 정했는데 이에 대한 의견이 궁금합니다.

아 맞다 클라이언트 압축 요청드리기로 했었죠..! 넵 일단 5mb로 하고 부족하거나 너무 크다고 생각되면 조정하면 될듯합니다,,!

  • test setup 파일에서 member 1을 admin, member 2를 그냥 study member, member 3을 외부자로 추가하면서 테스트 관련 코드가 조금 변경됐는데 이에 대해 의견이 있으시다면 공유해주시면 감사하겠습니다!

넵 기존 케이스가 깨지지 않는다면 문제 없을거같아요 👍

@github-actions github-actions bot added D-1 1일 전 까지 리뷰해주세요 and removed D-2 2일 전 까지 리뷰해주세요 labels Apr 22, 2024
Copy link
Collaborator Author

@05AM 05AM left a comment

Choose a reason for hiding this comment

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

리뷰 확인했습니다 👍

@05AM
Copy link
Collaborator Author

05AM commented Apr 23, 2024

  • soft delete에 대한 의견이 궁금합니다.

👍👍👍👍👍👍👍👍👍👍👍👍👍👍👍👍👍👍👍👍👍👍👍👍👍👍👍👍👍👍👍 💪💪💪💪💪💪💪💪💪💪💪💪💪💪💪💪💪

ㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋ 무서워요.... 😅😅😅

@github-actions github-actions bot added D-0 바로 리뷰가 필요해요 and removed D-1 1일 전 까지 리뷰해주세요 labels Apr 23, 2024
@05AM
Copy link
Collaborator Author

05AM commented Apr 25, 2024

@zxcv9203

리뷰 반영 완료하여 머지하겠습니다!

@05AM 05AM merged commit 9446197 into dev Apr 25, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
D-0 바로 리뷰가 필요해요
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants