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

Pagination 기능 구현 및 Books 페이지 개선 #148

Merged
merged 2 commits into from
Mar 3, 2024
Merged

Conversation

jihwooon
Copy link
Owner

@jihwooon jihwooon commented Mar 3, 2024

작업 내용 (Content)

링크 (Links)

기타 사항 (Etc)

Merge 전 필요 작업 (Checklist before merge)

  • PR 올리기 전 rebase 동기화를 하셨나요?
  • 마지막 줄에 공백 처리를 하셨나요?
  • 커밋 단위를 의미 단위로 나눴나요?
    • 예시
      • 코드 가독성을 위해 메서드를 추출하라
      • if-else 문을 if 문으로 분리하라
      • 불필요한 메서드를 인라인화하라
  • 커밋 본문을 작성하셨나요?
    • 예시
      • 함수는 한 가지 일을 해야 한다는 원칙에 따라 메서드를 추출합니다.
      • if-else는 컴파일 시 처리가 되어 재컴파일 없이 수정 할 수 없습니다.
        이에 따라 코드가 실행되는 순간에 실행이 결정되는 if 문으로 수정합니다.
  • CI 파이프라인이 통과가 되었나요?
  • 리뷰 요청 전 Self-Review로 의문점을 해결 하셨나요?
  • PR 리뷰 가능한 크기를 유지하셨나요?

jihwooon added 2 commits March 3, 2024 13:39
설명:
- Pagination 컴포넌트 구현:
  - 페이지네이션 기능을 담당하는 Pagination 컴포넌트가 완성되었습니다.
  - URLSearchParams를 활용하여 페이지 이동을 처리합니다.
  - 현재 페이지를 시각적으로 강조하며, 클릭 가능한 페이지 버튼을 표시합니다.
- Books 페이지 수정:
  - Books 페이지에서 Pagination 컴포넌트를 활용하여 페이지네이션을 표시합니다.
  - Pagination 컴포넌트에 필요한 pagination 데이터를 전달합니다.
- useBooks hook 수정:
  - Pagination에 필요한 pagination 데이터를 포함하도록 수정했습니다.
- Books 페이지 스타일링:
  - Books 페이지의 전체 레이아웃과 필터 영역 스타일을 조정했습니다.

관련 변경 사항:
- client/src/components/books/Pagination.tsx 파일 수정
- client/src/hooks/useBooks.ts 파일 수정
- client/src/pages/Books.tsx 파일 수정
설명:
- deploy.yml 파일 이름을 server.yml 로 변경했습니다.
- 이름 변경은 파일 내용보다 정확하게 반경하기 위해서 변경했습니다.

관련 변경 사항
- 파일명을 변경했습니다
@jihwooon jihwooon merged commit 3a13e10 into main Mar 3, 2024
1 check passed
@jihwooon jihwooon deleted the issue-76 branch March 3, 2024 04:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

1 participant