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

[UI/#214] 달력 / 2차 스프린트 #222

Merged
merged 17 commits into from
Sep 9, 2024
Merged

[UI/#214] 달력 / 2차 스프린트 #222

merged 17 commits into from
Sep 9, 2024

Conversation

boiledEgg-s
Copy link
Member

@boiledEgg-s boiledEgg-s commented Sep 7, 2024

⛳️ Work Description

  • UI 수정
  • 주간->월간 슬라이드 전환 추가
  • 토스트 추가
  • 빈화면 그래픽 추가
  • 서버 관련 코드 수정

📸 Screenshot

  • 빈화면 UI

  • 슬라이드 모션

    KakaoTalk_Video_2024-09-07-15-32-51.mp4

📢 To Reviewers

  • 달력 2차 스프린트 분량 구현이 거의 끝났습니다~ UI적으로 바뀐건 많이 없는데, UI를 바꾸다 보니까 통신 관련 코드도 같은 브랜치에서 바꾸게 되버렸네요,,ㅎㅎ
  • 다이얼로그를 연결한 코드도 마련되어 있으니까 참고해서 다이얼로그 붙이시면 될 것 같습니닷!
  • 여기에서 달력 전환과 관련된 수정이 더 이뤄질 예정입니다!!
    계획은 이 브랜치에서 자식 브랜치를 하나 파서 작업하고 순차적으로 머지할 예정입니다.
    해당 계획과는 별개로 이 PR엔 리뷰를 달아주시면 됩니다~!!

@boiledEgg-s boiledEgg-s added MOD ✅ 코드 수정 및 내부 파일 수정 UI 💐 UI 작업 석준💜 석준 labels Sep 7, 2024
@boiledEgg-s boiledEgg-s added this to the 2차 스프린트 작업 milestone Sep 7, 2024
@boiledEgg-s boiledEgg-s self-assigned this Sep 7, 2024
@boiledEgg-s boiledEgg-s requested a review from arinming September 8, 2024 15:26
Copy link
Member

@leeeyubin leeeyubin left a comment

Choose a reason for hiding this comment

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

확인했습니다!! 2차 스프린트까지 수고많았어요:) 짱!!

그리구 혹시 지금 소브렌치 파서 작업중인가용..?
오늘 apk 파일 보내려구 하는데 현재 올려둔 피알은 머지 못하는 거 아닌가 해서 물어봅니당..!

Comment on lines 25 to +27
fun NavGraphBuilder.calendarNavGraph(
navHostController: NavController
navHostController: NavController,
paddingValues: PaddingValues
Copy link
Member

Choose a reason for hiding this comment

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

반영 감사합니당~!!

Copy link
Member

@Hyobeen-Park Hyobeen-Park left a comment

Choose a reason for hiding this comment

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

우왕 수고하셨습니당~~~!!! 진짜 최고네욤😊

Copy link
Contributor

@arinming arinming left a comment

Choose a reason for hiding this comment

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

ㅜㅜ 그저 최고,, 2차 스프린트 수고하셨어요!!!!

@@ -37,7 +41,7 @@ fun NavGraphBuilder.calendarNavGraph(
}
) {
CalendarRoute(
navigateUp = navHostController::navigateUp,
modifier = Modifier.padding(paddingValues),
Copy link
Contributor

Choose a reason for hiding this comment

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

확인했음도!

@boiledEgg-s boiledEgg-s merged commit 61fa69b into develop Sep 9, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
MOD ✅ 코드 수정 및 내부 파일 수정 UI 💐 UI 작업 석준💜 석준
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[UI] 캘린더 / 2차 UI 수정
4 participants