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

Feature/main feeds api #38

Merged
merged 21 commits into from
Dec 15, 2024
Merged

Feature/main feeds api #38

merged 21 commits into from
Dec 15, 2024

Conversation

lee1nna
Copy link
Collaborator

@lee1nna lee1nna commented Dec 13, 2024

Pull Request 요약

PR에 대한 내용을 작성해주세요.

메인 피드 API 연동 작업


작업 내용

작업 내용에 대해서 작성해주세요.

  • 메인 피드 전체 조회 API 연동
  • 모듈화된 query key 상수 관리
  • 피드 무한 스크롤 구현
  • 피드 좋아요 API 연동
  • 피드 삭제 API 연동

참고 사항

참고 사항이 있다면 첨부해주세요.

서버측 피드 데이터가 없어 현재 mock.ts 파일에 있는 ALL_FEEDS 데이터를 임시로 넣어둔 상태입니다.
메인 피드에 대한 리팩토링은 서버측에서 피드데이터가 추가되면 별도의 브랜치로 진행 할 예정입니다.


관련 이슈

closing할 이슈 번호를 작성해주세요.


@lee1nna lee1nna added the feat 구현, 개선 사항에 관련된 이슈입니다. label Dec 13, 2024
@lee1nna lee1nna self-assigned this Dec 13, 2024
Copy link

netlify bot commented Dec 13, 2024

Deploy Preview for prostargram failed.

Name Link
🔨 Latest commit 762a4ce
🔍 Latest deploy log https://app.netlify.com/sites/prostargram/deploys/675d3ae821ab9c0008be205c

Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
12 Security Hotspots
6.5% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube Cloud

Copy link
Collaborator

@seongjin2427 seongjin2427 left a comment

Choose a reason for hiding this comment

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

고생 많으셨습니다!! 👍

url: `/comments/${commentId}/likes`,
headers: {
Authorization:
'Bearer eyJhbGciOiJIUzUxMiJ9.eyJhdXRob3JpdHkiOiJOT1JNQUxfVVNFUiIsInRva2VuVHlwZSI6IkFDQ0VTU19UT0tFTiIsInN1YiI6IjEiLCJleHAiOjE3NDI3MTUzMDV9.eFVccnbsYRqpJFzg2rL5LNgsMLoEFPESoj3W4e5bgm2Uzy7-DCjA6hgixDZ8MYgXeV6q7VetKQTab-pH-g0caw',
Copy link
Collaborator

Choose a reason for hiding this comment

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

동일한 테스트용 토큰을 활용하신다면
별도의 파일에 변수로 분리해서 import하면 더 좋을 것 같습니다. 😅

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

나중에 다 지워질 부분이라 일단 저렇게 냅뒀는데, 리팩토링 시 참고하겠습니다 !!

@lee1nna lee1nna merged commit 432221d into develop Dec 15, 2024
1 of 6 checks passed
@lee1nna lee1nna deleted the feature/main-feeds-api branch December 16, 2024 06:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat 구현, 개선 사항에 관련된 이슈입니다.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

메인 피드 API 연동
2 participants