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

Fix/pagination #26

Open
wants to merge 3 commits into
base: dev
Choose a base branch
from
Open

Conversation

ye-yo
Copy link
Contributor

@ye-yo ye-yo commented Mar 5, 2022

PR 타입(하나 이상의 PR 타입을 선택해주세요)

  • 기능 추가
  • 기능 삭제
  • 버그 수정
  • 의존성, 환경 변수, 빌드 관련 코드 업데이트

반영 브랜치

dev <- fix/pagination

변경 사항

  1. 디바이스 화면에서 페이지네이션 영역이 넘치는 문제 존재 > 페이지네이션 버튼 레이아웃에 반응형 적용
  • 기존 <Button>은 style 수정이 안된다고 해서 <TouchableOpacity>로 컴포넌트 변경
  1. 페이지네이션 10페이지 이상 넘어가지 않는 문제
    최대 페이지 수 계산하여 전달 (numberOfpages={maxPageNum})
    저장소 검색은 10개 issue는 5 페이지씩 뜨도록 pageLimit props 추가
    항상 10개/5개씩 보여주는 것이 아닌 남은 페이지 만큼 보여주도록 수정(getPageLength)
  2. 페이지네이션에 따라 data slice 시 start, end index 수정

테스트 결과

Issue

  • 잦은 저장소 검색시 403 오류 발생. 기다리다보면 결과가 나타나는 경우도 있음(현재는 throw error해서 error로 판단하도록 함)
  • Fix: Pagination 로직 및 레이아웃 #25 의 4번째 문제는 미해결.
  • 내비게이션 버튼 클릭시 페이지네이션의 현재 페이지 초기화필요

테스트하면서 우선적으로 해결이 가능한 이슈들만 해결했습니다.
이슈 페이지 페이지네이션은 좀 더 고민이 필요할 것 같습니다.
기존 페이지네이션 코드에서 수정한 부분이 있기때문에 검토해보시고 의도와 다른 부분이나 잘못된 부분 말씀부탁드립니다.

테스트 화면

테스트화면

resolved: #25

@ye-yo ye-yo added the bug Something isn't working label Mar 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Fix: Pagination 로직 및 레이아웃
2 participants