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.

Sign up for GitHub

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

Jump to bottom

[#87] Feat: 짤 이미지 신고 확인 모달 UI 구현 #88

Merged
merged 10 commits into from
Mar 6, 2024
Merged

[#87] Feat: 짤 이미지 신고 확인 모달 UI 구현 #88

merged 10 commits into from
Mar 6, 2024

Conversation

HY219
Copy link
Contributor

@HY219 HY219 commented Mar 3, 2024 •
edited
Loading

📝 작업 내용

  • 짤 이미지 신고 확인 모달 UI 구현
  • 짤 이미지 상세 페이지에서 신고 아이콘 클릭 시 모달 열기

📷 스크린샷 (선택)

💬 리뷰 요구사항(선택)

📍 기타 (선택)

  • UI 확인 시 ./src/routes/_layout-with-chat/index.tsx 파일의 7번째 줄에 다음 코드를 추가해주세요.

    <ImageDetailModal isOpen={true} onClose={() => {}} />

  • 신고된 사진 제거 확인 모달 UI 코드 참고하여 구현
  • TODO: 짤 이미지 신고 API 구현 코드와 연결 및 API 요청 성공/실패 시 동작 구현

close #87

@HY219
Feat: 짤 이미지 신고 확인 모달 UI 구현
b1f5078 
신고된 사진 제거 확인 모달 UI 코드 참고하여 구현
TODO: 짤 이미지 신고 api 연결 및 성공/실패 시 동작 구현
@HY219 HY219 self-assigned this Mar 3, 2024
@vercel Vercel
Copy link

vercel bot commented Mar 3, 2024 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
zzalmyu-frontend ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 5, 2024 6:11pm

@HY219 HY219 added the ✨ Feature 새로운 기능 추가 label Mar 3, 2024
@Kim-Jaemin420
Copy link
Contributor

@HY219 서버의 cloudFront 배포 id가 변경됐거나 수정사항이 있는 것 같아요! 오늘 아침에 확인해보겠습니다!

@Kim-Jaemin420
Copy link
Contributor

@HY219 cli로 확인해본 결과, 백엔드에서 넘겨준 cloudFront id가 유효하지 않네요
스크린샷 2024-03-04 오전 4 31 13

Kim-Jaemin420
Kim-Jaemin420 reviewed Mar 3, 2024
View reviewed changes
src/components/ReportConfirmModal/index.tsx Outdated Show resolved Hide resolved
src/components/ReportConfirmModal/index.tsx Outdated Show resolved Hide resolved
src/components/ReportConfirmModal/index.tsx Outdated Show resolved Hide resolved
src/components/ReportConfirmModal/index.tsx Outdated Show resolved Hide resolved
src/components/ImageDetailModal/ImageMenuBar.tsx Outdated Show resolved Hide resolved
@HY219
Modify: ModalHeader 컴포넌트의 children Props를 옵셔널로 변경
2306a90 
전달한 children props가 없을 경우 Modal.Header를 좀 더 간결하게 작성
@HY219
Rename: 컨벤션과 역할에 맞게 이벤트 핸들러 함수 이름 수정
296deb3 
handleClickReport -> handleOpenReportConfirmModal
handleReport -> handleClickReport
줄 간격 띄움
@HY219
Refactor: hasCloseButton props에서 true 값 삭제
9e8ea4a 
hasCloseButton props의 값은 동적으로 변경되지 않고 고정값 true를 가집니다.
@HY219
Refactor: import 및 hook 호출 순서 변경
4603d10 
useState를 useOverlay 앞에 배치
코드의 가독성 향상 및 일관성 유지
@HY219
Rename: 짤 이미지 상세모달에서 신고 버튼 클릭 시 호출되는 이벤트 핸들러 함수 이름 변경
cbba680 
handleOpenReportConfirmModal -> handleClickReportButton
어떤 동작을 하는 함수인지에 대한 설명이 아닌, 클릭했을 때 발생하는 동작을 설명해주는 이름이므로 변경
@HY219
Merge branch 'main' of https://github.com/zzalmyu/zzalmyu-frontend into
ffd1b47 
#87/feature/report-confirmation-modal-ui
Kim-Jaemin420
Kim-Jaemin420 approved these changes Mar 5, 2024
View reviewed changes
Copy link
Contributor

@Kim-Jaemin420 Kim-Jaemin420 left a comment

Choose a reason for hiding this comment

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

LGTM!👍

yjc2021
yjc2021 approved these changes Mar 6, 2024
View reviewed changes
Copy link
Contributor

@yjc2021 yjc2021 left a comment

Choose a reason for hiding this comment

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

LGTM입니다 👍

@HY219 HY219 merged commit 48a201c into main Mar 6, 2024
3 checks passed
@HY219 HY219 deleted the #87/feature/report-confirmation-modal-ui branch March 6, 2024 00:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@choi-ik choi-ik choi-ik left review comments

@yjc2021 yjc2021 yjc2021 approved these changes

@Kim-Jaemin420 Kim-Jaemin420 Kim-Jaemin420 approved these changes

@hyeonjinan096 hyeonjinan096 Awaiting requested review from hyeonjinan096 hyeonjinan096 is a code owner

Assignees

@HY219 HY219

Labels
✨ Feature 새로운 기능 추가
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

신고 확인 모달 UI 구현
5 participants
@HY219 @Kim-Jaemin420 @yjc2021 @choi-ik @hyeonjinan096