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

[ISSUE-99&100] 약속 잡기 STEP 3 - 캘린더 구현 & 캘린더를 공통 컴포넌트로 분리 #101

Merged
merged 7 commits into from
Jul 22, 2022

Conversation

hoyahozz
Copy link
Member

@hoyahozz hoyahozz commented Jul 19, 2022

ISSUE


작업 내용

  • 캘린더 데코레이터 리팩터링
  • 약속 잡기 STEP 3의 캘린더 구현 및 ViewState 분리
  • 기존 사용하던 캘린더를 공통 컴포넌트로 분리
  • 추가로, ViewModel 에서 스트링 리소스를 사용할 수 있도록 확장 클래스를 추가하였습니다!
    • 메시지가 다른 스낵바를 출력해야 할 때, 사이드이펙트 두 개를 만드는 것보다 메시지를 매개변수로 받아 확장성을 늘리는 것이 더 합리적이라고 판단하여 확장 클래스를 생성하게 되었습니다~!
    • 혹시 의견 있으시면 언제든지 말씀해주세요!

실행 화면

Screen Shot Screen Shot
Screenshot_1658191133 Screenshot_1658191136
Screenshot_1658191138

Check List

  • PR 제목은 [ISSUE-{N}] PR 제목으로 작성
  • CI/CD 통과 여부
  • 1명 이상의 Approve 확인 후 Merge
  • 관련 이슈 연결

@hoyahozz hoyahozz added this to the Sprint #7 milestone Jul 19, 2022
@hoyahozz hoyahozz self-assigned this Jul 19, 2022
@hoyahozz hoyahozz merged commit b90024f into develop Jul 22, 2022
@hoyahozz hoyahozz deleted the feature/issue-99-calendar branch July 22, 2022 15:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Sprint #7] 캘린더를 공통 컴포넌트로 분리 [Sprint #7] 약속잡기(P_03) - 캘린더 추가
1 participant