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: 커서 페이징 적용 #364

Merged
merged 11 commits into from
Feb 22, 2024
Merged

feat: 커서 페이징 적용 #364

merged 11 commits into from
Feb 22, 2024

Conversation

anyukyung
Copy link
Member

🧑‍🚀 PR 요약

해당 pr에서 작업한 내역을 적어주세요.

  • 홈, 홈->재생, 프로필에서 커서 페이징을 적용했습니다.
  • 신고 접수 후 문구를 변경했습니다.
  • 프로필 편집시 피커를 사용할 때 발생하는 버그를 고쳤습니다.

📌 변경 사항

변경사항 및 주의 사항 (모듈 설치 등)을 적어주세요.

  • 프로필 이미지를 이미지 캐싱하도록 변경 (그래서 Data로 변환하여 전달하는 부분 삭제)
  • posts를 얻어올 때 페이지네이션이 필요한 모델을 추가했습니당
struct PostsPage {
    let cursor: Int
    let posts: [Post]
}
📸 ScreenShot

작동, 구현화면

RPReplay_Final1708414599.MP4
RPReplay_Final1708414610.MP4

Linked Issue

close #361

Copy link
Collaborator

@chopmozzi chopmozzi left a comment

Choose a reason for hiding this comment

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

고생하셨습니다.
캐시하는 부분 추가 됐다는게 lo.setImage하는 부분 맞나요?

Comment on lines +178 to +179
//
// func fetchPosts(at cursor: Int?, of id: Int?) async -> PostsPage? {
Copy link
Collaborator

Choose a reason for hiding this comment

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

얘는 이제 사라져도..?

@anyukyung
Copy link
Member Author

고생하셨습니다. 캐시하는 부분 추가 됐다는게 lo.setImage하는 부분 맞나요?

@chopmozzi 넵 맞습니당 그래서 worker에서 데이터로 바꿔주는 부분도 없앴습니다!

@anyukyung anyukyung merged commit 0347086 into iOS/dev Feb 22, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants