Skip to content

[Feat/#39] main page publishing 및 api 연동#45

Merged
hyeonjin6530 merged 27 commits intodevelopfrom
feature/#39-main-page-publishing
Feb 24, 2025
Merged

[Feat/#39] main page publishing 및 api 연동#45
hyeonjin6530 merged 27 commits intodevelopfrom
feature/#39-main-page-publishing

Conversation

@hyeonjin6530
Copy link
Member

@hyeonjin6530 hyeonjin6530 commented Feb 23, 2025

📌 관련 이슈번호

🎟️ PR 유형

어떤 변경 사항이 있나요?

  • 새 기능 추가
  • 버그 수정
  • CSS 등 사용자 UI 디자인 변경
  • 리팩토링

Check List

  • 🔀 PR 제목의 형식을 잘 작성했나요?
  • 🏗️ yarn build는 성공했나요?
  • 🧹 불필요한 코드는 제거했나요? (TODO, 주석, clg... etc.)
  • ✅ 컨벤션을 지켰나요?
  • 💭 이슈는 등록했나요?
  • 🏷️ 라벨은 등록했나요?
  • 🙇‍♂️ 리뷰어를 지정했나요?

✅ Key Changes

이번 PR에서 작업한 내용을 간략히 설명해주세요

< 퍼블리싱 >

  1. 메인 페이지 퍼블리싱
  2. 대여하기 bottom sheet 컴포넌트 제작
  3. 메인 페이지용 헤더 제작
  4. 검색 기능(디바운스)
  5. 광고 캐러셀

< API 연동 >

  1. 본인의 반납 필요 대여 기록 조회
  2. 본인의 대여 기록 조회
  3. 대여 취소
  4. 반납 신청
  5. 물품 대여 신청
  6. 대여 물품 목록 조회
  7. 알림 개수 조회
  8. 광고(노션 연결)

+) 추가적인 기능 구현

  1. s3로부터 이미지 받아올 수 있도록 설정
  2. isAdmin으로 sidemenu 관리자 메뉴 유무 결정
  3. 대여 불가 뱃지 추가(REJECTED 상태)
  4. 반납 대기 중일 때 취소하기 기능 삭제
  5. 기존의 type으로 인한 error 해결

📢 To Reviewers

  • 급하게 하느라 한 브랜치에 와랄라 작업 진행했습니다!

  • api의 경우, type을 따로 정의하고 swagger에 나누어져 있는 파트 기준으로 파일을 만들어 data 받아오는 부분만 분리하여 작성하였습니다.

    • 예시
      image
      image
  • 파일 마다 axios를 import 하고 토큰도 일단 test 토큰 정의하여 사용중이라서 나중에 custom axios 파일을 만들어서 리팩토링 진행하면 될 거 같습니다! (토큰 제외)

  • main에 있는 carousel에 들어갈 이미지를 cover로 할지 contain 중 어떤 것으로 하는 게 좋을까요?(일단은 contain으로 할 경우 빈공간이 생길 수 있어서 cover로 해두었습니다.)

📸 스크린샷

  • 메인 페이지
image
  • 바텀시트
    imageimage

🔗 참고 자료

@hyeonjin6530 hyeonjin6530 added 🚀 API API ✨ FEAT 기능 구현 labels Feb 23, 2025
@hyeonjin6530 hyeonjin6530 self-assigned this Feb 23, 2025
@hyeonjin6530 hyeonjin6530 merged commit 4f84f87 into develop Feb 24, 2025
2 checks passed
@sinji2102 sinji2102 deleted the feature/#39-main-page-publishing branch March 5, 2025 12:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🚀 API API ✨ FEAT 기능 구현

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[ Feat ] 복지물품 대여 페이지 퍼블리싱 및 api 연동

2 participants