Skip to content

Conversation

@cozy-ito
Copy link
Collaborator

@cozy-ito cozy-ito commented May 3, 2025

#️⃣연관된 이슈

ex) #이슈번호, #이슈번호

Closes #70

📝 PR 유형

해당하는 유형에 'x'로 체크해주세요.

  • 기능 추가 (Feature)
  • 버그 수정 (Bug Fix)
  • 코드 개선 (Refactoring)
  • 스타일 변경 (UI/UX)
  • 문서 작업 (Documentation)
  • 환경 설정 (Configuration)
  • 기타 (Other)

📝작업 내용

이번 PR에서 작업한 내용을 간략히 설명해주세요(이미지 첨부 가능)

컴포넌트 관련

  • NoticeEmployerPage 컴포넌트 UI 구현
  • NoticeDetailInfo 컴포넌트 (+ 사장님 버전까지 확장)
  • ShopInfoPostCardSkeleton -> NoticeDetailSkeleton 컴포넌트명 변경

그 외 변경 사항

  • 수빈님 요청 (@almighty55555): 공고 편집 라우터 경로 /notice/edit -> /notice/edit/:noticeId로 변경
  • 도메인 NOTICE 관련 로직 src/apis/loaders/notice.ts 파일로 분리
  • src/utils/localStorage.ts 유틸 함수 구현 (getLocalStorageValue, setLocalStorageValue)
  • 최근 본 공고 리스트 제어를 위한 커스텀 훅 구현 및 적용 (useUpdateRecentNotices. 로컬 스토리지 사용)

기타

zustand 스토어는 컴포넌트 외부에서도 접근이 가능하네요!? (신기)
(컴포넌트 내에서 사용하는 것 같이 상태가 변경되어도 자동으로 반응하지는 않는대요 🤔)

// src/pages/NoticeEmployerPage/loader/noticeEmployerLoader.ts
const noticeEmployerLoader: LoaderFunction = async ({ params }) => {
  const user = useUserStore.getState().user; // useUserStore.getState()로 바로 state 참조 가능
  const { shopId, noticeId } = params as {
    shopId: string;
    noticeId: string;
  };
  ...
};

코드 리뷰 후 변경사항

  • loader 로직 리팩토링
  • APPLICATION_STATUS 상수화
  • NoticeDetailInfoCard 컴포넌트 버튼 분리 (NoticeEmployerActionButton, NoticeEmployeeActionButton)
  • useShopApplications 커스텀 훅 의존성 배열 요소 추가
  • 최근 본 공고 리스트 나타나지 않는 버그 수정

스크린샷 (선택)

💬리뷰 요구사항(선택)

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

ex) 메서드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?

@cozy-ito cozy-ito added the Feature (기능 추가) 기능을 추가합니다. label May 3, 2025
@github-actions github-actions bot requested review from almighty55555 and ghdtnals May 3, 2025 15:49
@netlify
Copy link

netlify bot commented May 3, 2025

Deploy Preview for thejulge1 ready!

Name Link
🔨 Latest commit a67167e
🔍 Latest deploy log https://app.netlify.com/sites/thejulge1/deploys/6816fe3cae7f3a00081eceba
😎 Deploy Preview https://deploy-preview-73--thejulge1.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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

Copy link
Collaborator

@almighty55555 almighty55555 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

@ghdtnals ghdtnals left a comment

Choose a reason for hiding this comment

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

수고히셨습니다!👍

@cozy-ito cozy-ito merged commit a5ab8fd into dev May 4, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Feature (기능 추가) 기능을 추가합니다.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

공고 상세 페이지 (사장님) (/notice/{shopId}/{noticeId}/employer)

4 participants