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 [#199] Spotify API 아티스트 조회 시 아이디가 50개 넘어가면 나눠서 조회하도록 변경 #200

Merged
merged 4 commits into from
Jan 24, 2025

Conversation

ch1hyun
Copy link
Collaborator

@ch1hyun ch1hyun commented Jan 24, 2025

✨ Issue Number ✨

closes #199

✨ To-do ✨

  • 아이디가 50개 넘어가면 나눠서 조회하도록 변경

✨ Description ✨  

image image

SpotifyAPIHandler 에 아티스트 아이디 여러 개로 조회하는 함수를 추상화를 했습니다.
해당 함수에서는 아티스트 아이디가 Spotify API 가 허용하는 아이디의 최대 개수 (현재는 50개)를 넘는지 확인하고, 넘는다면 나눠서 조회한 뒤 한 번에 모아 반환합니다.

@ch1hyun ch1hyun added the 🐞 BugFix 버그 수정 label Jan 24, 2025
@ch1hyun ch1hyun self-assigned this Jan 24, 2025
Copy link
Collaborator

@Ivoryeee Ivoryeee left a comment

Choose a reason for hiding this comment

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

확인했숩니다 굿굿

@ch1hyun ch1hyun merged commit 5d45d60 into develop Jan 24, 2025
1 check passed
@ch1hyun ch1hyun deleted the fix#199/devideRequestArtist branch January 24, 2025 09:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐞 BugFix 버그 수정
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[fix] Spotify API 아티스트 조회 시 아이디가 50개 넘어가면 나눠서 조회하도록 변경
2 participants