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

Feat [#39] challenge 부분 UI 및 분기처리 추가 #41

Merged
merged 11 commits into from
May 30, 2024

Conversation

kim-seonwoo
Copy link
Member

👾 작업 내용

  • 접기 펼치기 부분을 구현하였습니다!
  • 분기처리로 아이콘들을 구별하여 상태에 따라 다른 아이콘이 오게 하였습니다.

🚀 PR Point

  • 작업 겹치는 부분 살리는 과정에서 둘 다 다른 기능이면 살렸습니다.!
  • challengeWeekView부분이 특히 전부 겹쳐서 일단 제 브랜치걸로 해놨습니다만,,
    그라데이션 코드만 가지고 40번 브랜치 코드로 해도 좋습니다!!
  • 겹치는 부분이 있다보니까 문제 있으신거 말씀해주시면 바로 고칠게요!!

📸 스크린샷

구현 내용 스크린샷
챌린지 Simulator Screen Recording - iPhone 13 mini - 2024-05-29 at 17 36 07

✅ CheckList

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

🔗 Issue

Resolved #39

@kim-seonwoo kim-seonwoo added 😎선우 선우의 issue ✨FEAT 기능 구현 labels May 29, 2024
@kim-seonwoo kim-seonwoo added this to the 🔭2차 스프린트🪐 milestone May 29, 2024
@kim-seonwoo kim-seonwoo requested a review from Zoe0929 May 29, 2024 10:26
@kim-seonwoo kim-seonwoo self-assigned this May 29, 2024
@kim-seonwoo kim-seonwoo linked an issue May 29, 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.

굳!

Circle()
.foregroundColor(.clear) // 내부를 채우는 원
.frame(width: 44, height: 44)
VStack(alignment: .leading) {
Copy link
Member

Choose a reason for hiding this comment

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

요 부분 사실 제가 비슷하게 짰다가 뷰 안에 요소가 너무 많아서 컴파일이 불가하다는 에러가 발생했어요! 이거는 제 브랜치 머지 후 ViewBuilder 사용한 코드로 바꾸면 좋을 것 같아요

@kim-seonwoo kim-seonwoo merged commit 0014322 into develop May 30, 2024
@Zoe0929 Zoe0929 deleted the feat/#39-ChallengeView branch June 19, 2024 02:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨FEAT 기능 구현 😎선우 선우의 issue
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[Feat] ChallengeView Header 구현
2 participants