[F-BAN] 신고하기 모달 #430
[F-BAN] 신고하기 모달 #430
Conversation
src/components/ReportModal.tsx
Outdated
| const onSubmit: SubmitHandler<IReportFormInput> = (data) => { | ||
| //console.log(data) | ||
| setIsSubmitting(true) | ||
| axiosInstance | ||
| .post(`api/v1/report`, data) | ||
| .then((res) => { | ||
| console.log(res) | ||
| successOpenToast() | ||
| handleClose() | ||
| }) | ||
| .catch((error) => { | ||
| console.log(error.message) | ||
| failedOpenToast() | ||
| }) | ||
| setIsSubmitting(false) | ||
| } |
There was a problem hiding this comment.
코드를 보면서 생각해봤는데 로그인을 한 상태로만 신고를 하게 해야할지, 지금처럼 모든 유저가 신고를 할 수 있게 해야할 지 고민이 되네요. 혹시 백엔드랑 논의해보셨나요?
There was a problem hiding this comment.
일단 로그인 상태에서만 신고가 가능하게 되어있습니다. (로그인 상태가 아닌상태에서 최종 신고하기를 버튼을 누르면 로그인 화면으로 이동)
src/components/ReportModal.tsx
Outdated
| const { | ||
| CuToast: SuccessToast, | ||
| isOpen: isSuccessOpen, | ||
| openToast: successOpenToast, | ||
| closeToast: successCloseToast, | ||
| } = useToast() | ||
| const { | ||
| CuToast: FailedToast, | ||
| isOpen: isFailedOpen, | ||
| openToast: failedOpenToast, | ||
| closeToast: failedCloseToast, | ||
| } = useToast() |
There was a problem hiding this comment.
상황에 따라 다른 toast를 보여줘야 해서 2개의 훅을 쓰신거죠...?
일단 최신 dev 브랜치에 아마 useToast의 상태 중에 toastMessage가 추가되었을거예요! 이걸로 메시지별로 toast 컴포넌트를 만들지 않고도 여러 개의 메시지를 보여주게 할 수 있을 것 같습니당
그리고 지금 코드는 toast의 severity가 달라지는 경우인데 이것도 위에 toastMessage 추가한 것 처럼 훅 안에 severity state를 추가해서 그걸 바꿔주는 식으로 해서 하나의 toast 컴포넌트로 처리하게 만들 수 있을 것 같아요!
src/components/ReportModal.tsx
Outdated
| let typeName: string | ||
|
|
||
| switch (reportType) { | ||
| case 'user': | ||
| typeName = '유저를' | ||
| break | ||
| case 'showcase': | ||
| typeName = '쇼케이스를' | ||
| break | ||
| case 'recruit': | ||
| typeName = '모집글을' | ||
| break | ||
| } |
There was a problem hiding this comment.
reportType에 따라 typeName을 바꿔주는 거라면 이런 식으로 지정해줘도 좋을 것 같아요
// 이거를 컴포넌트 밖에 선언해두고
const typeName = {
user: '유저를',
showcase: '쇼케이스를',
recruit: '모집글을',
}
// 아래에서는 typeName[reportType] 으로 접근!
|
@KRimwoo 컨플릭트 수정 후 멘션 주시면 승인 바로 드리겠습니다. |
|
|
@HiHoi 코멘트 반영해서 코드 수정했습니다~! |
작업내용
기타 사항