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/261 #367

Merged
merged 5 commits into from
Jun 17, 2024
Merged

Feat/261 #367

merged 5 commits into from
Jun 17, 2024

Conversation

devkyoung2
Copy link
Member

@devkyoung2 devkyoung2 commented Jun 14, 2024

#️⃣연관된 이슈

close: #261

📝작업 내용

  • 내 펀딩 아이템 조회 api 연동
    image
  • 펀딩 취소 정상 동작 확인
  • 펀딩 상품이 없을 경우 default 컴포넌트 렌더링
    image

🙏리뷰 요구사항(선택)

  • 펀딩 친구 랭킹은 예전에 제가 구현해놓긴 했는데 응답 형식에 맞게 수정하고 병합하려 했어서 stash된 상태입니다. 그래서 api 연동 이슈 따로생성하지 않고 기존에 작업중이었던 #120에서 api연동까지 같이 처리하겠습니다.
  • 해당 pr 이 머지된 이후 친구 랭킹 api 연동 작업을 이어서 할 수 있을것같아요! 빠른 확인 부탁드립니다.

@devkyoung2 devkyoung2 self-assigned this Jun 14, 2024
Copy link

vercel bot commented Jun 14, 2024

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

Name Status Preview Comments Updated (UTC)
kakao-funding ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 14, 2024 9:20am

Copy link
Contributor

@uraflower uraflower left a comment

Choose a reason for hiding this comment

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

👍

Comment on lines +14 to 16
const isEmptyObject = (obj: object): boolean => {
return Object.keys(obj).length === 0;
};
Copy link
Contributor

Choose a reason for hiding this comment

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

#368 에서 이 함수를 추출하셨는데, 현재 PR에서는 추출되지 않은 상태로 사용하고 있어서, dev에 병합할 때 이 코드가 추출되지 않은 채 살아 있을 수도 있을 것 같아요.
#368 PR 병합하기 전에 이 PR이 병합되어야 할 것 같습니다. 혹시나 꼬이거나 할까봐 말씀드립니다..!!

@devkyoung2 devkyoung2 merged commit 5ec10dd into dev Jun 17, 2024
3 checks passed
@devkyoung2 devkyoung2 deleted the feat/261 branch June 18, 2024 10:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

펀딩아이템 페이지 - 펀딩아이템 조회 API 연동
3 participants