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

[FEAT] 타임 블록 api 수정 #196

Merged
merged 65 commits into from
Dec 13, 2024
Merged

[FEAT] 타임 블록 api 수정 #196

merged 65 commits into from
Dec 13, 2024

Conversation

Chan531
Copy link
Contributor

@Chan531 Chan531 commented Dec 12, 2024

✨ Related Issue

📝 기능 구현 명세

image
image

  • 타임 블록 생성 및 태그 데이터 생성 확인

image
image

  • 타임 블록 파일 태그 추가 및 데이터 생성 확인

image
image

  • 타임 블록 파일 태그 삭제 및 데이터 삭제 확인

image
image

  • 파일 휴지통 이동으로 인한 데이터 삭제 확인

image
image

  • 타임 블록 삭제로 인한 데이터 삭제 확인

🐥 추가적인 언급 사항

  • 이전 pr에 언급을 못했는데 @ResponseStatus을 통해 코드를 좀 더 간결하게 작성해보았습니다. 이런 방식으로 하면 201일 때 URI를 안 담아줘도 돼서 전에 앱잼 때 언급하셨던 코드 복잡성이 완화된다는 장점이 있습니다. 좋은 거 같다면 전체적으로 적용해볼라고요!
  • 저번에 다대다 관계를 관리하는 도메인의 이름이 길어져 복잡성이 생긴다는 말씀에 공감하여 이번에 앞글자만 따서 클래스를 만들어보았습니다. DB에서는 풀네임이 들어가고 어떻게 생각하시는지 의견 남겨주시면 좋을 거 같아요.
  • 타임 블록 api에 변경사항이 상당히 많아 수정된게 좀 많습니다. 천천히 확인해주세요!
  • s3 파일 삭제가 url 말고 key라는 것으로 이루어진다는 것을 테스트하다가 확인했습니다. 따라서 팀 icon image도 url과 key로 관리해야 삭제할 수 있을 거 같아요. 다음 pr 때 적용 부탁드립니다!

파일 id 리스트를 받도록 수정
orElseThrow 제거
orElseThrow 제거
@Chan531 Chan531 added the feat 기능 label Dec 12, 2024
@Chan531 Chan531 requested a review from paragon0107 December 12, 2024 07:27
@Chan531 Chan531 self-assigned this Dec 12, 2024
Copy link
Contributor

@paragon0107 paragon0107 left a comment

Choose a reason for hiding this comment

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

수고하셨습니닷
주석 부분 코멘트 관련해서 남겨주시고 수정사함 없을것 같으면 바로 머지해주세여

@Chan531 Chan531 merged commit 7dc8a09 into develop Dec 13, 2024
1 check passed
@Chan531 Chan531 deleted the feat/#194-time-block branch December 13, 2024 06:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat 기능
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FEAT] 타임 블록 API 수정
2 participants