Skip to content

[Feature/#58] : 캘린더 약속 정보 가져오기 API 연결 #70

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.

Sign up for GitHub

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

Jump to bottom
Merged
youjin09222 merged 27 commits into from
Mar 24, 2025
Merged

[Feature/#58] : 캘린더 약속 정보 가져오기 API 연결 #70

youjin09222 merged 27 commits into from
Mar 24, 2025

Conversation

@youjin09222
Copy link
Contributor

@youjin09222 youjin09222 commented Mar 21, 2025 •
edited
Loading

✅ 𝗖𝗵𝗲𝗰𝗸-𝗟𝗶𝘀𝘁

  • merge할 브랜치의 위치를 확인해 주세요.(main❌/develop⭕)
  • 리뷰가 필요한 경우 리뷰어를 지정해 주세요.
  • 리뷰는 PR이 올라오면 최대한 빠르게 진행합니다.
  • P1 단계의 리뷰는 빠르게 확인 후 반영합니다.
  • Approve된 PR은 assigner가 머지하고, 수정 요청이 온 경우 수정 후 다시 push를 합니다.

📌 𝗜𝘀𝘀𝘂𝗲𝘀

📎 𝗪𝗼𝗿𝗸 𝗗𝗲𝘀𝗰𝗿𝗶𝗽𝘁𝗶𝗼𝗻

  • ScheduleItem 가로 정렬 수정
  • 캘린더 약속 정보 가져오기 API 연결
  • 현재 월 변경 시 캘린더 API 재호출
  • 바텀 시트에 캘린더 데이터 적용

📷 𝗦𝗰𝗿𝗲𝗲𝗻𝘀𝗵𝗼𝘁

KakaoTalk_20250324_223108610.mp4

💬 𝗧𝗼 𝗥𝗲𝘃𝗶𝗲𝘄𝗲𝗿𝘀

캘린더 해결 됐습니다ㅏㅏ

캘린더 정보 받아오긴하는데 빈 리스트로 들어와서 서버에 연락 드렸습니다.
(+ 약속 확정 단계에서 문제 발생해서 해결 후 연락 주신다고 하셨습니다!)
image

youjin09222 added 21 commits March 3, 2025 01:34
@youjin09222
#58 [MOD] ScheduleItem 레이아웃 fillMaxWidth 적용하여 가로 정렬 수정
464b879
@youjin09222
Merge branch 'develop' of https://github.com/Noostak/Noostak_Android …
f0b8855 
…into feature/#58-calendar-appointments-api
@youjin09222
Merge branch 'develop' of https://github.com/Noostak/Noostak_Android …
e02aa4e 
…into feature/#58-calendar-appointments-api
@youjin09222
#58 [ADD] ApiKeyStorage에 CALENDAR 추가
0507612
@youjin09222
#58 [ADD] ApiKeyStorage에 YEAR, MONTH 추가
3694484
@youjin09222
#58 [ADD] ResponseGetCalendarDto 추가
29a2106
@youjin09222
#58 [FEAT] CalendarApiService 구현
3fb2ba1
@youjin09222
#58 [ADD] CalendarEntity 추가
3bcc5fb
@youjin09222
#58 [ADD] ResponseGetCalendarDtoMapper 추가
d7812ae
@youjin09222
#58 [FEAT] CalendarDataSource 추가
9e6aa14
@youjin09222
#58 [FEAT] CalendarDataSourceImpl 구현
e0f30c4
@youjin09222
#58 [FEAT] CalendarRepository 추가
7bc6672
@youjin09222
#58 [FEAT] CalendarRepositoryImpl 구현
051315d
@youjin09222
#58 [FEAT] DataSourceModule에 CalendarDataSource 바인딩 추가
fa1c8bc
@youjin09222
#58 [FEAT] RepositoryModule에 CalendarRepository 바인딩 추가
f605596
@youjin09222
#58 [FEAT] ServiceModule에 provideCalendarService 추가
7e1f44f
@youjin09222
#58 [FEAT] GetCalendarAppointmentsUseCase 구현
384f514
@youjin09222
#58 [DEL] 캘린더 더미 데이터 삭제
cd62947
@youjin09222
#58 [FEAT] 캘린더 약속 정보 가져오기 API 연동
a32eda7
@youjin09222
#58 [FEAT] 현재 월 변경 시 캘린더 API 재호출
ef07fbe
@youjin09222
#58 [DEL] getLocalDateByPage 확장 함수 제거
c72426d
@youjin09222 youjin09222 added ⭐ [FEAT] 새로운 기능 구현 🐰 유진 뉴진스 labels Mar 21, 2025
@youjin09222 youjin09222 self-assigned this Mar 21, 2025
@youjin09222 youjin09222 requested a review from a team as a code owner March 21, 2025 11:58
gaeulzzang
gaeulzzang approved these changes Mar 23, 2025
View reviewed changes
Copy link
Contributor

@gaeulzzang gaeulzzang left a comment

Choose a reason for hiding this comment

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

현재 자동로그인 코드가 빠져 있는 것 같습니다!! saveIsAutoLogin인가 그거 뷰모델에 추가해주세여
캘린더 정보는 언젠가 들어오겠져~ 일단 성공한거니까 어푸 합니다 ^^

Eonji-sw
Eonji-sw reviewed Mar 24, 2025
View reviewed changes
Copy link
Member

@Eonji-sw Eonji-sw left a comment

Choose a reason for hiding this comment

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

LGTM
자동로긘 제가 한줄 추가했는데 제 pr 확인해보시고 맞으면 걍 추가안하고 머지해도 될듯요?

@youjin09222
#58 [FIX] 캘린더 일정 키 포맷 yyyy-MM-dd로 변경
2f9e4ec
@youjin09222
Merge branch 'develop' of https://github.com/Noostak/Noostak_Android …
4977e18 
…into feature/#58-calendar-appointments-api

# Conflicts:
#	app/src/main/java/com/sopt/noostak/di/DataSourceModule.kt
#	app/src/main/java/com/sopt/noostak/di/RepositoryModule.kt
#	app/src/main/java/com/sopt/noostak/di/ServiceModule.kt
@youjin09222 youjin09222 merged commit f829be3 into develop Mar 24, 2025
1 check passed
@gaeulzzang gaeulzzang deleted the feature/#58-calendar-appointments-api branch April 1, 2025 05:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Eonji-sw Eonji-sw Eonji-sw approved these changes

@gaeulzzang gaeulzzang gaeulzzang approved these changes

Assignees

@youjin09222 youjin09222

Labels

⭐ [FEAT] 새로운 기능 구현 🐰 유진 뉴진스

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[FEAT] : 캘린더 약속 정보 가져오기 API 연결

4 participants

@youjin09222 @Eonji-sw @gaeulzzang