Skip to content

feat ✨ follow feed #419

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.

Sign up for GitHub

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

Jump to bottom
Open
niamu01 wants to merge 13 commits into
base: dev
Choose a base branch
from
Open

feat ✨ follow feed #419

niamu01 wants to merge 13 commits into from

Conversation

@niamu01
Copy link
Contributor

@niamu01 niamu01 commented Apr 10, 2024 •
edited
Loading

리베이스... 순서 헷갈려서 되돌릴지 고민하다 체리픽으로 브랜치 다시 만들었습니다... 🥺

저번 일요일(4/7) 회의 :

  • pagination 방식 변경
    • (전) pagination을 위해 모두 로드하기 -> (후) totalCount를 없애고 날짜 기준 로드 & ttl 적용
    • feed db에서 모두 가져온 후 팔로우 여부는 js로 필터링
  • 다중서버환경에서의 정합성 문제 해결
  • 실험실 기능처럼 일부 먼저 업데이트

조금 더 생각해보니:

  • 점점 양이 많아질텐데 모든 feed를 preload하는건 별론거같당 🤔
  • 최근 한달간의 feed 로드 & ttl 설정 => cron을 통해 주기적 캐시 생성

niamu01 added 11 commits April 10, 2024 19:16
@niamu01
feat: ✨ 임시 데이터 사용을 위해 temp_follows로 변경
6c790a5 
- #416
@niamu01
feat: ✨ feed 형식 제안
f0cb0d3 
- #416
@niamu01
refactor: ♻️ feedbase로 분류 (type은 각 피드 성격에 맞게 고정)
7ff37db 
- #416
@niamu01
feat: ✨ cursorPagination 추가, follow관련 피드 테스트
1c93af5 
- #416
@niamu01
feat: ✨ 반환값에 맞에 pagination 되도록 구현, follow 제외 다른 피드 항목들 mock data 반환하도…
ab69a13 
…록 추가

- #416
@niamu01
refactor: 🐛 값이 0일때를 고려해 null처리 변경 || -> ??, push에서 스프레드연산자로 변경
d1403e8 
- #416
@niamu01
refactor: ♻️ FeedPaginationed -> FeedPaginated, db 도큐먼트 생성 날짜 created…
c4c54fd 
…At로 통일

- 3416
@niamu01
feat: ✨ follow 이벤트 발생 시 feed db에도 추가
219e2ef 
- #416
@niamu01
refactor: ♻️ discriminatorKey 시도
4465769 
- #416
@niamu01
feat: ✨ 요청 시 feed db 에서 반환
8f4f3d1 
- #416
@niamu01
refactor: ♻️ 배열 합치는 부분 flat으로 변경
af1e3fc 
flatMap은 빈 배열을 합치지 못하더라구용

- #416
@niamu01 niamu01 changed the base branch from main to dev April 10, 2024 10:36
@niamu01 niamu01 added the enhancement New feature or request label Apr 10, 2024
@niamu01 niamu01 self-assigned this Apr 10, 2024
niamu01 added 2 commits April 12, 2024 20:17
@niamu01
feat: ✨ 최근 1달간의 피드 캐시 추가
e4d1045 
- #416
@niamu01
feat: ✨ totalCount가 없는 pagination 제작
1eee93f 
틀만 갖추도록 만들어 두고 endCursor도 없어질 예정

- #416
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@niamu01 niamu01

Labels

enhancement New feature or request

Projects

42Stat
Status: 👀 In review

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@niamu01