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] 페이징 예외 수정 #72

Merged
merged 1 commit into from
Feb 17, 2024
Merged

[Fix] 페이징 예외 수정 #72

merged 1 commit into from
Feb 17, 2024

Conversation

Jeongmin39
Copy link
Contributor

✨ PR 유형

어떤 변경 사항이 있나요??

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

🛠️ 작업내용

getPageOfPlaces() 메서드에서 startIndex가 places 리스트 크기를 초과할 경우 발생하는 Index Out of Bounds 예외 수정

📋 추후 진행 상황

📌 리뷰 포인트

✅ Checklist

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

  • 커밋 메시지 컨벤션에 맞게 작성했습니다
  • 유지-보수를 위해 주석 처리를 잘 작성했습니다

@Jeongmin39 Jeongmin39 added 🚶‍♀️정민 정민이 담당 🔥 bug 버그 수정 labels Feb 17, 2024
@Jeongmin39 Jeongmin39 self-assigned this Feb 17, 2024
Copy link
Member

@arinming arinming left a comment

Choose a reason for hiding this comment

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

수고하셨어욤 빠르게 머지하면 좋을 것 같습니당!

Copy link
Contributor

@sanggae4133 sanggae4133 left a comment

Choose a reason for hiding this comment

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

LGTM

@arinming arinming merged commit 8ce8551 into dev Feb 17, 2024
2 checks passed
@arinming arinming deleted the feature/34 branch March 10, 2024 10:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔥 bug 버그 수정 🚶‍♀️정민 정민이 담당
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants