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

✨ 피드 신고 UI #57

Merged
merged 29 commits into from
May 14, 2024
Merged

✨ 피드 신고 UI #57

merged 29 commits into from
May 14, 2024

Conversation

BangDori
Copy link
Collaborator

@BangDori BangDori commented May 14, 2024

작업 이유

  • 피드 신고 UI 구현

작업 사항

1️⃣ 피드 신고 UI

  1. 신고 카테고리
    • useReportCategories hook 관리
    • 신고 카테고리에 대한 상수 추가
  2. 신고 사유 - useInput hook 관리
  3. 숨김 처리 체크박스 - useToggle hook 관리
  4. 아이콘 추가 (checkbox-circle_off, 'checkbox-circle_on', 'checkbox-square_on', 'checkbox-square_off')

하나의 컴포넌트에서 하나의 관심사를 가질 수 있도록 컴포넌트들을 제작하였습니다.

  • FeedKebabButton: 케밥 메뉴 버튼으로 버튼을 클릭하였는지에 대한 관심사만을 가집니다.
  • KebabMenu: 케밥 메뉴 리스트로 게시물 숨기기 혹은 신고하기 버튼을 클릭하였는지에 대한 관심사만을 가집니다.
  • FeedReportsForm: 피드 신고 양식 컴포넌트로 피드 신고 양식과 관련한 모든 정보를 담당합니다.
    • useReportCategories: 피드 신고 카테고리를 관리하는 훅스입니다.
    • useInput: 신고 사유를 관리하는 공용 훅스입니다.
    • useToggle: 블라인드 처리를 관리하는 공용 훅스입니다.

2️⃣ useHooks 공용 훅 구현

  • useToggle
  • useInput

리뷰어가 중점적으로 확인해야 하는 부분

  • 📌 개인적으로 피드 신고 양식 폼에 부족한 점이 많다고 생각합니다! 개선 사항 혹은 의문점이 드시는 부분에 대해 적극적으로 질문 부탁드립니다!

  • 발생한 이슈에 대해 인지하셨나요?


발견한 이슈

@BangDori BangDori self-assigned this May 14, 2024
Copy link

This pull request is automatically being deployed by Amplify Hosting (learn more).

Access this pull request here: https://pr-57.d37mn03xh3qyyz.amplifyapp.com

Copy link
Member

@Legitgoons Legitgoons left a comment

Choose a reason for hiding this comment

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

수고 많으셨습니다. hooks도 잘 분리하셨고, 카테고리를 따로 변수로 분리해서 처리하는 점도 좋았습니다. 병합하셔도 좋습니다.

src/widgets/feed-kebab/model/useVisibilityKebabMenu.tsx Outdated Show resolved Hide resolved
src/features/feed-reports/ui/FeedReportsForm.tsx Outdated Show resolved Hide resolved
@BangDori BangDori merged commit 194a420 into main May 14, 2024
2 checks passed
@BangDori BangDori deleted the feature/PW-321-feed-reports branch May 14, 2024 14:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants