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.

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

[FEAT] 메인페이지 페이징작업 #393

Merged
merged 14 commits into from
May 27, 2023
Merged

[FEAT] 메인페이지 페이징작업 #393

merged 14 commits into from
May 27, 2023

Conversation

dlrjswns
Copy link
Contributor

📌 PR 요약

🌱 작업한 내용

  • 메인페이지 게시글 및 투두리스트 커서페이징 작업
  • 게시글 및 투두리스트 생성에 따른 데이터 최신화

🌱 PR 포인트

  • 기존에 커서페이징에 대해 잘못 알게된 개념이 있어서 홈, 검색 등등 커서페이징이 적용되는 모든 부분을 다시 전면 수정하여 올릴 예정입니다.
  • 게시글의 갯수가 많아짐에 따라서 셀의 재사용 문제로 승현님께서 만든 BoardCollectionViewCell에 prepareForReuse함수를 구현하였는데 혹여 기존 동작에 문제가 생기면 말씀해주세요
  • @WhiteHyun 그리고 profileImage가 nil로 내려올때 게시글의 프로필이미지 디폴트이미지로 변경시키는 코드도 추가시켜줘야할꺼같아요

📸 동작영상

Simulator.Screen.Recording.-.iPhone.14.Pro.-.2023-05-25.at.22.34.26.mp4

📮 관련 이슈

dlrjswns added 13 commits May 22, 2023 23:14
@dlrjswns
[FEAT] 스크롤 끝에 닿을 시 데이터 요청 로직 추가
c8262b5
@dlrjswns
[FEAT] 메인페이지 게시판에 대한 데이터 요청 및 필터 로직 추가
96548ed
@dlrjswns
[FEAT] 메인페이지 투두리스트 추가 데이터 요청 및 필터 로직 추가
517f031
@dlrjswns
[DEL] 테스트 목데이터 모델 삭제
82ac824
@dlrjswns
[FEAT] 투두리스트, 게시판 페이징 작업 및 생성 시 데이터최신화 작업 추가
8a64549
@dlrjswns
[ADD] 피그마디자인을 위한 UI 추가
e9d37e8
@dlrjswns
[FEAT] 페이징 및 데이터 최신화를 위한 로직 수정
5e00e76
@dlrjswns
[ADD] TodoGoalHeaderView 파일 추가
6928b36
@dlrjswns
[CHORE] 전체 스크롤을 위한 컬렉션뷰로 코드 수정
5d29c40
@dlrjswns
[FEAT] 게시글 커서 페이징 및 코드 수정
e58f6bd
@dlrjswns
[FEAT] 투두리스트 커서 페이징 구현 및 코드 수정
2621cf0
@dlrjswns
[ADD] 게시글셀에 대한 데이터 재사용 코드 추가
bebdc45
@dlrjswns
[DEL] 필요없는 코드 삭제
4d2741c
@dlrjswns dlrjswns added 😎 Enhancement New feature or request D+0 꼭 오늘 안에 PR 리뷰 해주세요 🥳MainPage labels May 26, 2023
@dlrjswns dlrjswns requested review from WhiteHyun and soobin-k May 26, 2023 04:20
@dlrjswns dlrjswns self-assigned this May 26, 2023
WhiteHyun
WhiteHyun approved these changes May 26, 2023
View reviewed changes
Copy link
Member

@WhiteHyun WhiteHyun left a comment

Choose a reason for hiding this comment

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

꽤나 복잡하네요 ㄷㄷ.. 고생하셨습니다.

언급하신 부분은 수정하겠습니다!

soobin-k
soobin-k approved these changes May 26, 2023
View reviewed changes
Copy link
Contributor

@soobin-k soobin-k left a comment

Choose a reason for hiding this comment

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

고생많으셨습니다 건준님

@dlrjswns dlrjswns merged commit f4baab9 into develop May 27, 2023
@dlrjswns dlrjswns deleted the feat/Mainpage-341/Paging branch May 27, 2023 14:55
@dlrjswns dlrjswns mentioned this pull request May 27, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@WhiteHyun WhiteHyun WhiteHyun approved these changes

@soobin-k soobin-k soobin-k approved these changes

Assignees

@dlrjswns dlrjswns

Labels
D+0 꼭 오늘 안에 PR 리뷰 해주세요 😎 Enhancement New feature or request 🥳MainPage
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@dlrjswns @WhiteHyun @soobin-k