[QA 중간점검] 셰어, 마인더 편지 이슈 해결

[rmdnps10]

Checklist

• 올바른 브랜치에 PR을 보내도록 설정하였나요?
• PR의 라벨을 올바르게 달았나요?

Description

PR 내용을 작성해주세요.

Related Issues

1. 셰어 편지 작성 페이지에서 카테고리 선택 안하고 제출하기 누를 시 현재 URL유지 ([Refactor] 셰어 편지 작성에서, 상담 카테고리 선택 안할 시 현재 페이지 유지 #278)
2. 마인더 편지 작성에서 임시저장 시에 제출하기, 임시저장 버튼 활성화
3. 마인더 편지 작성에서 OngoingConsultBox 스타일 오류 해결
4. 항상 5분 전으로 나와있던 편지 시간 api 응답값으로 수정

To Reveiwer

리뷰어가 주목해야할 사항들을 작성해주세요

[netlify]

✅ Deploy Preview for candid-semolina-d0db42 ready!

Name	Link
🔨 Latest commit	0bda485
🔍 Latest deploy log	https://app.netlify.com/sites/candid-semolina-d0db42/deploys/6633a626bd5e380008369d4a
😎 Deploy Preview	https://deploy-preview-279--candid-semolina-d0db42.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[kyuhho]

예전에 작성한 코드라 undefined 를 허용하거나 null 과 함꼐 혼용한 케이스가 많은 거 같습니다!

interface OngoingCounsultBoxProps {
  categoryStatus?: string;
  consultStatus?: string;
  counselorName: string | undefined;
  beforeMinutes: string | undefined | null;
  content: string | null | undefined;
  newMessageCounts: number | null;
  counselorprofileStatus: number | undefined;
  onClick?: () => void;
  reviewCompleted?: boolean;
  isChat?: boolean;
}

그리고 ? 를 붙여주지 않아도 되는 부분에서 모두 ? 를 붙여준 부분들이 많이 보입니다!

            counselorName={consultInform.counselorName}
            categoryStatus={consultInform.categoryStatus}
            beforeMinutes={consultInform.beforeMinutes}
            counselorprofileStatus={consultInform.counselorprofileStatus}
            content={consultInform.content}

지금 당장은 어쩔 수 없지만, 앞으로 구현되거나 고치는 부분에서는 api 스펙에 맞게 null 이 허용된 property만 | null 을 사용하고, state를 initailize하고 싶을 땐, 전부 undefined 로 선언하기 보단 default 값을 설정해주면 좋을 것 같습니다!

 const [consultInform, setConsultInform] = useState<LetterConsultInform>({
    categoryStatus: undefined,
    counselorName: undefined,
    beforeMinutes: undefined,
    content: undefined,
    newMessageCounts: undefined,
    counselorprofileStatus: undefined,
    date: undefined,
  });

오류 처리 방식에 대해 깔끔한 코드일수록 유지보수가 쉬워지는 거 같아요! 제 코드도 고칠 부분이 많아 하나씩 고쳐나가면 좋을 것 같습니다😂😂
console.log 만 지워주세요!

[kyuhho]

console.log는 지워주세요~!