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: 검색 자동 완성 페이징 로직 수정 #14

Merged
merged 2 commits into from
Jan 29, 2024
Merged

Conversation

Go-Jaecheol
Copy link
Contributor

Issue

✨ 구현한 기능

  • 검색 자동 완성 API에서 hasNext 관련 로직 수정
  • 관련 검증 테스트 추가

📢 논의하고 싶은 내용

  • X

🎸 기타

  • 자바 버전 올린 기념으로 .toList()로 바꿔봤는데, 다른 메서드들에서도 쓴 거 다 바꾸면 코드 변경 많아질 거 같아서 이건 나중에 이슈 파서 한 번 수정하죠~

⏰ 일정

  • 추정 시간 : 30min
  • 걸린 시간 : 10min

Copy link

Test Results

0 tests   0 ✅  0s ⏱️
0 suites  0 💤
0 files    0 ❌

Results for commit c044b57.

Copy link
Contributor

@wugawuga wugawuga left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!

@Go-Jaecheol Go-Jaecheol merged commit 4b33265 into develop Jan 29, 2024
3 checks passed
@Go-Jaecheol Go-Jaecheol deleted the fix/issue-13 branch January 29, 2024 11:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

검색 자동 완성 페이징 로직 수정
2 participants