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

Feat: 실제 API 연동 및 혼합 접근법 활용 상태 관리 추가 #51

Merged
merged 6 commits into from
Dec 9, 2024

Conversation

sunglitter
Copy link
Collaborator

🧃 Pull Request

🍉 코멘트

  • 관리자 계정 접속 시 Admin Page로 이동 및 하위 항목 추가
  • 실제 API 연동 및 상태 관리(혼합 접근법 활용 전역 상태 관리 방식 추가)
  • 이미지 프리뷰 함수 별도 생성 및 분리, 관련 페이지 내 활용

🪿 연관된 이슈

✅ PR 포인트 & 궁금한 점

  • 늦어서 죄송합니다 ㅠㅠ... 아직 기능 정상 작동 여부 확인 중입니다...

- 관리자 계정으로 접속한 경우 My Page 대신 Admin Page로 접속하도록 수정
- Admin Page 하위 항목에 게시물 관리 및 채팅방 관리 추가
- 권한에 따른 페이지 전환 로직 및 네비게이션 구조 개선
- 전체 코드에서 실제 API 연동 작업 완료
- 혼합 접근법 활용 전역 상태 관리 방식 추가(jotai + react-query)
- 포스트 상세 조회 페이지 컴포넌트 구조 분리 및 리팩토링
- 환불 진행 및 환불 완료 페이지 파일명만 생성(빈 파일 상태)
- 이미지 프리뷰 처리 로직을 별도 함수로 생성 및 분리
- 분리된 프리뷰 함수 재사용으로 코드 중복 제거
- 이미지 업로드 기능이 있는 페이지들에 분리된 함수 적용
@sunglitter sunglitter added the feat feat label Dec 9, 2024
@sunglitter sunglitter self-assigned this Dec 9, 2024
Copy link
Collaborator

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

@OpenS3same OpenS3same left a comment

Choose a reason for hiding this comment

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

고생많으셨습니다!

- 이미지 프리뷰 관련 함수를 별도 유틸리티 폴더로 이동
- 시간 관련 조정 함수 생성 및 관련 페이지에 반영
- 관리자 승인 대기 목록에서 마감일 정보 숨김 처리
- 게시글 승인 페이지 라우터 경로 수정으로 네비게이션 동작 개선
@sunglitter sunglitter added the refactor refactor label Dec 9, 2024
@sunglitter sunglitter merged commit 55850a1 into main Dec 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat feat refactor refactor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants