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

상품 개수 API 연결 #33

Merged
merged 4 commits into from
Feb 3, 2024
Merged

상품 개수 API 연결 #33

merged 4 commits into from
Feb 3, 2024

Conversation

WhiteHyun
Copy link
Member

Screenshots 📸

테스트 화면
Screen Recording 2024-02-02 at 12 21 53 PM



고민, 과정, 근거 💬

Home에서 사용하는 API가 다수 존재하여, HomeEndPoint의 타입을 enum으로 변경했습니다.
URLProtocol도 다수 API를 처리할 수 있도록 dictionary를 설정했습니다.


@WhiteHyun WhiteHyun added 🏠 Home Home View 🔨 Implementation New feature or request labels Feb 2, 2024
@WhiteHyun WhiteHyun added this to the v2.0.0 milestone Feb 2, 2024
@WhiteHyun WhiteHyun requested a review from a team February 2, 2024 03:25
@WhiteHyun WhiteHyun self-assigned this Feb 2, 2024
Copy link
Contributor

@eung7 eung7 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다! 🤩

eung7

This comment was marked as duplicate.

@WhiteHyun WhiteHyun requested a review from eung7 February 2, 2024 12:09
@WhiteHyun WhiteHyun merged commit 1240c2d into main Feb 3, 2024
6 checks passed
@WhiteHyun WhiteHyun deleted the feature/home/32 branch February 3, 2024 01:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🏠 Home Home View 🔨 Implementation New feature or request
Projects
Development

Successfully merging this pull request may close these issues.

행사 제품의 개수를 보여준다.
2 participants