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]#71 feature 공통 기능 리펙토링 및 포킷내 링크 개수 표시관련 수정 #74

Merged
merged 9 commits into from
Sep 22, 2024

Conversation

l5x5l
Copy link
Contributor

@l5x5l l5x5l commented Sep 20, 2024

Key Changes

feature 모듈들에서 공통적으로 사용되던 페이지네이션을 core:feature 모듈로 분리하여 중복 로직을 제거했습니다.
링크를 수정할 때 속한 포킷을 변경하게 되어 포킷 내 링크 개수가 변할때 이를 반영하지 못한 문제를 수정했습니다.
Resolves: #71

PR 유형

어떤 변경 사항이 있나요?

  • 새로운 기능 추가
  • 버그 수정
  • CSS 등 사용자 UI 디자인 변경
  • 코드에 영향을 주지 않는 변경사항(오타 수정, 탭 사이즈 변경, 변수명 변경)
  • 코드 리팩토링
  • 주석 추가 및 수정
  • 문서 수정
  • 테스트 추가, 테스트 리팩토링
  • 빌드 부분 혹은 패키지 매니저 수정
  • 파일 혹은 폴더명 수정
  • 파일 혹은 폴더 삭제

To Reviewers

  • 앞으로 페이징 관련 기능 구현할 때마다 core:feature에 SimplePaging 클래스를 사용하려 하는데, 혹시 코드 보면서 헷갈리거나 바로 이해가 안되는 부분 있다면 사소한거라도 피드백 부탁해!
  • 그 외 다른 부분도 지적 환영!

PR Checklist

PR이 다음 요구 사항을 충족하는지 확인하세요.

  • 커밋 메시지 컨벤션에 맞게 작성했습니다.
  • 정해진 코딩 컨벤션에 맞게 작성했습니다.
  • 변경 사항에 대한 테스트를 했습니다.(버그 수정/기능에 대한 테스트)

Etc.

@l5x5l l5x5l linked an issue Sep 20, 2024 that may be closed by this pull request
3 tasks
Copy link
Member

@jiwon2724 jiwon2724 left a comment

Choose a reason for hiding this comment

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

고생하셨습니당 -1,281 맛있ㄷ ㅏ ~ ㅎ

@l5x5l l5x5l merged commit a05a8ba into develop Sep 22, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FIX] 일부 코드 리펙토링 및 세부 내용 수정
2 participants