Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Network [#43] point 네트워크 구현, 토스트 메세지 #46

Merged
merged 9 commits into from
May 31, 2024

Conversation

kim-seonwoo
Copy link
Member

👾 작업 내용

  • 포인트 관련 네트워크를 구현하였습니다.
  • 포인트 관련 Api를 PointRouter로 구현하였어요!
  • customAlert 관련 메모리 이슈를 해결하였습니다... (진땀승)
  • 챌린지 완료 시 alert를 연결하였습니다.
  • 토스트 메세지를 추가하고 연결하였어요!
  • 회원가입 실패시 토스트 메세지가 뜨도록 변경

🚀 PR Point

  • "포인트를 다 받은 후 챌린지를 생성하세요" 이 부분의 토스트 메세지는 아직 연결을 안했습니다.
    .showToast(toastType: .onboardingWarn, isPresented: $onboardingViewModel.isOnboardingError)
    다음과 같이 enum 값과 binding bool 값을 이용하여 viewModifier로 선언하면 됩니다.

📸 스크린샷

구현 내용 스크린샷
화면종류 image

✅ CheckList

  • 오류 없이 빌드되는지 확인
  • 로그용 print문 제거
  • 불필요한 주석 제거
  • 코드 컨벤션 확인

🔗 Issue

Resolved #이슈번호

@kim-seonwoo kim-seonwoo added 😎선우 선우의 issue 🎨STYLE 커스텀 뷰 짜기 📡NETWORK API 작업 labels May 30, 2024
@kim-seonwoo kim-seonwoo added this to the 🔭2차 스프린트🪐 milestone May 30, 2024
@kim-seonwoo kim-seonwoo requested a review from Zoe0929 May 30, 2024 15:31
@kim-seonwoo kim-seonwoo self-assigned this May 30, 2024
@kim-seonwoo kim-seonwoo linked an issue May 30, 2024 that may be closed by this pull request
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.

👍 진심으로 감사드립니다.. 최고 최고 !

@kim-seonwoo kim-seonwoo merged commit c3be176 into develop May 31, 2024
@Zoe0929 Zoe0929 deleted the network/#43-Point branch June 19, 2024 02:22
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
📡NETWORK API 작업 🎨STYLE 커스텀 뷰 짜기 😎선우 선우의 issue
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[Network] 포인트 뷰 api
2 participants