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

Feature/#378 팀 이미지 삭제 #379

Merged
merged 3 commits into from
Jan 26, 2025

Conversation

yeonddori
Copy link
Collaborator

관련 이슈

작업 요약

  • 팀 수정 시 이미지 삭제도 가능하도록 아이콘을 추가하였습니다.

작업 상세 설명

  • 삭제 아이콘 버튼은 팀 이미지 수정 아이콘 오른쪽에 배치하였습니다.
  • api 동작은 아직 확인을 못 해봤습니다. 추후에 확인해 본 후 다시 수정하겠습니다!

리뷰 요구 사항

  • IconBox 컴포넌트에서 아이콘 크기를 40px로 직접 지정하여서, 수정 모달의 팀 썸네일 영역에서도 <Box w="368px">로 고정된 너비를 사용했는데, 직접 px을 주는 방식 외에 더 유연한 방법이 있는지 잘 모르겠습니다.. 100%를 잘 활용하고 싶었는데 마음처럼 잘 안되더라고요😥

미리 보기

image

@yeonddori yeonddori added the ✨ 기능 개발 새로운 기능을 구현하는 데 필요한 작업 또는 변경 사항 label Jan 12, 2025
@yeonddori yeonddori self-assigned this Jan 12, 2025
Copy link
Collaborator

@llddang llddang left a comment

Choose a reason for hiding this comment

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

image

현재 삭제 관련하여서 문제가 있는 것 같습니다
확인 부탁드립니다! :)

@yeonddori
Copy link
Collaborator Author

image 현재 삭제 관련하여서 문제가 있는 것 같습니다 확인 부탁드립니다! :)

서버 설정을 수정한 이후로 이제 제대로 작동합니다!

@yeonddori yeonddori requested a review from llddang January 26, 2025 10:45
Copy link
Collaborator

@llddang llddang left a comment

Choose a reason for hiding this comment

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

굿입니다!

Copy link
Collaborator

@jasper200207 jasper200207 left a comment

Choose a reason for hiding this comment

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

👍

@yeonddori yeonddori merged commit 924bc2f into develop Jan 26, 2025
1 check passed
@yeonddori yeonddori deleted the Feature/#378-팀_이미지_삭제 branch January 26, 2025 13:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ 기능 개발 새로운 기능을 구현하는 데 필요한 작업 또는 변경 사항
Projects
None yet
Development

Successfully merging this pull request may close these issues.

팀 이미지 삭제
3 participants