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 [#91] MyPage 서버 통신 중복 수정 #98

Merged
merged 1 commit into from
Jan 16, 2024

Conversation

boyeon0119
Copy link
Contributor

👾 작업 내용

  • 마이페이지에서 getUserDataAPI()를 헤더뷰에서도 하고 마이페이지뷰에서도 해서 두 번 통신되는걸 수정했습니다.

🚀 PR Point

  • GOTCHA~

📸 스크린샷

구현 내용 스크린샷
통신 한번!

✅ Issue

Resolved #91

@boyeon0119 boyeon0119 added 🙊보연 보연의 issue 🐞fix 오류 고치는 것 / 수정용 📡 network api 작업 labels Jan 16, 2024
@boyeon0119 boyeon0119 added this to the 🚀1차 스프린트🚀 milestone Jan 16, 2024
@boyeon0119 boyeon0119 self-assigned this Jan 16, 2024
Copy link
Member

@Zoe0929 Zoe0929 left a comment

Choose a reason for hiding this comment

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

LGTM ~ 굳굳입니당

Copy link
Member

@kim-seonwoo kim-seonwoo left a comment

Choose a reason for hiding this comment

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

수고하셨습니다!

@@ -150,7 +150,6 @@ final class UserPointHeaderView: UICollectionReusableView {
}

private func configureView() {
getUserDataAPI()
Copy link
Member

Choose a reason for hiding this comment

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

p5. 오 잘 잡아내셨네요!!

@boyeon0119 boyeon0119 merged commit 3763f0c into develop Jan 16, 2024
@Zoe0929 Zoe0929 deleted the fix/#91-fixNetworkDuplication branch January 19, 2024 14:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐞fix 오류 고치는 것 / 수정용 📡 network api 작업 🙊보연 보연의 issue
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[Fix] 마이페이지 서버 통신 수정
3 participants