Skip to content

[#80] Feat/내프로필페이지 무한스크롤#96

Merged
eunkyung01 merged 9 commits intomainfrom
feat/내프로필페이지_무한스크롤
Feb 16, 2025

Hidden character warning

The head ref may contain hidden characters: "feat/\ub0b4\ud504\ub85c\ud544\ud398\uc774\uc9c0_\ubb34\ud55c\uc2a4\ud06c\ub864"
Merged

[#80] Feat/내프로필페이지 무한스크롤#96
eunkyung01 merged 9 commits intomainfrom
feat/내프로필페이지_무한스크롤

Conversation

@eunkyung01
Copy link
Collaborator

@eunkyung01 eunkyung01 commented Feb 16, 2025

#️⃣ Issue

📝 Description

  • 내 프로필 페이지의 fetch 함수들을 따로 분리하여 lib 폴더에 모아두었습니다!
  • 내가 쓴 후기 리스트 / 내가 등록한 와인 리스트에 무한 스크롤 기능을 구현해두었습니다.
    limit는 5로 설정되어있고, 아래로 내릴때마다 추가로 5만큼 더 불러옵니다!
  • 기존 alert부분은 Toast로 바꾸어두었습니다! (와인 혹은 리뷰 수정 완료, 삭제, 실패 / 프로필 수정 완료, 닉네임 중복, 수정 실패)
  • 스켈레톤 UI 로딩 처리 완료
  • 이외에는 자잘 버그 수정이 있습니다. (리뷰삭제하고 count 숫자에 반영되도록 수정, 닉네임이 30글자를 넘어가면 수정 못하게 막아둠)

npm run dev 하신후 로그인 하시면 내 프로필 페이지에서 바로 확인이 가능합니다!

🌀 PR Type

어떤 변경 사항이 있나요?

  • 새로운 기능 추가
  • 버그 수정
  • CSS 등 사용자 UI 디자인 변경
  • 코드에 영향을 주지 않는 변경사항 (오타 수정, 탭 사이즈 변경, 변수명 변경)
  • 코드 리팩토링
  • 주석 추가 및 수정
  • 문서 수정
  • 테스트 추가, 테스트 리팩토링
  • 빌드 부분 혹은 패키지 매니저 수정
  • 파일 혹은 폴더명 수정
  • 파일 혹은 폴더 삭제

✅ Checklist

PR Checklist

PR이 다음 요구 사항을 충족하는지 확인하세요.

  • Branch Convention 확인

    feat/ 피쳐, fix/ 버그 수정, refactor/ 개선

  • Base Branch 확인
  • 커밋 메시지 컨벤션 준수
  • 적절한 Label 지정
  • Assignee 및 Reviewer 지정

Test Checklist

  • 로컬 작동 확인

Additional Notes

  • 유저 이미지가 null값일때 유저 정보 PATCH를 하면 서버에서 400에러를 보내주는 것 처리하기.

@eunkyung01 eunkyung01 added Priority: Medium 🔖 우선순위 보통 Type: Feature ✨ 신규 기능 labels Feb 16, 2025
@eunkyung01 eunkyung01 requested a review from LMS10 February 16, 2025 11:27
@eunkyung01 eunkyung01 self-assigned this Feb 16, 2025
@vercel
Copy link

vercel bot commented Feb 16, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
wine ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 16, 2025 11:39am

@eunkyung01 eunkyung01 force-pushed the feat/내프로필페이지_무한스크롤 branch from 92278f6 to 5e2571b Compare February 16, 2025 11:32
Copy link
Owner

@LMS10 LMS10 left a comment

Choose a reason for hiding this comment

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

테스트 완료했습니다~! 수고하셨습니다 👍🏻

@eunkyung01 eunkyung01 merged commit c46ce8b into main Feb 16, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Priority: Medium 🔖 우선순위 보통 Type: Feature ✨ 신규 기능

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Feat : [내 프로필 페이지] 무한 스크롤 구현 및 코드 리팩토링

2 participants