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

[TNT-210] 트레이너 캘린더 api 연결 #83

Merged
merged 2 commits into from
Feb 13, 2025
Merged

Conversation

syss220211
Copy link
Member

📌 What is the PR?

  • 트레이너 캘린더 api를 연결하였습니다.

🪄 Changes

  • 달력 스케쥴 개수 표시에 필요한 데이터 요청 api 연결완료
  • 특정 날짜의 pt 리스트 불러오기 api 연결완료

🌐 Common Changes

🔥 PR Point

// Calendar
            VStack(spacing: 12) {
                TCalendarView(
                    selectedDate: $store.selectedDate,
                    currentPage: $store.view_currentPage,
                    events: store.events
                )
                .onChange(of: store.state.selectedDate, { oldValue, newValue in
                    let startOfDay = Calendar.current.startOfDay(for: newValue)
                    store.selectedDate = startOfDay
                    store.send(.view(.calendarDateTap))
                })
                .padding(.horizontal, 20)
            }
  • calendar에 selectedDate가 binding reducer에서 변화가 되지 않아서 이렇게 구현했습니다만.. 어떤점이 문제인지 잘 모르겠습니다. 추가로 해당하는 달에만 api 연결해놓아서 스크롤 또는 캘린더 헤더로 넘어가는 쪽은 추가구현할 예정입니다... 민서님 작업 부분이랑 겹치는 부분이 꽤 있는것 같아서 먼저 올렸어요!
  • 네비게이션 연결 전입니다.

🙆🏻 To Reviewers

💭 Related Issues

@FpRaArNkK FpRaArNkK self-requested a review February 13, 2025 07:26
Copy link
Contributor

@FpRaArNkK FpRaArNkK left a comment

Choose a reason for hiding this comment

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

고생하셨습니다! 홈 관련 나머지 작업하실때 멘션 남겨놓은 부분 반영햊 시면 좋을 것 같아요!
저 onChange부분은 저도 한번 체크해보겠릅니다

Comment on lines +188 to +189
let dateFormatter: DateFormatter = DateFormatter()
dateFormatter.dateFormat = "yyyy-MM-dd"
Copy link
Contributor

Choose a reason for hiding this comment

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

홈 관련 나머지 작업하실 때 저희 사용하고 있는 TDateFormatUtility로 바꿔주시면 좋을 것 같아요! DateFormatter 하나씩 만드는게 초기화 비용이 크더라구요

Copy link
Member Author

Choose a reason for hiding this comment

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

네~

@syss220211 syss220211 merged commit f1a1fa4 into develop Feb 13, 2025
@syss220211 syss220211 added the ✨Feat 새로운 기능 구현 (새로운 로직 추가, UI 구현 등) label Feb 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨Feat 새로운 기능 구현 (새로운 로직 추가, UI 구현 등)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[TNT-210] 트레이너 홈 화면 작성
2 participants