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

[Feature] 열람실 이용정보를 서버에 업로드합니다. #49

Merged
merged 16 commits into from
Oct 28, 2024

Conversation

Seokki-Kwon
Copy link
Contributor

@Seokki-Kwon Seokki-Kwon commented Oct 27, 2024

AS-IS

열람실 이용정보를 서버에 업로드합니다.

TO-BE

열람실 이용정보를 서버에 업로드합니다.

KEY-POINT

  • 메인화면의 경우 인터랙션이 복잡하기 떄문에 사전작업으로 컴포넌트 구조화 작업을 진행했습니다.
  • 스터디세션 인터랙터에 관련된 비즈니스 로직을 작성했습니다.
  • 열람실 이용관련은 전역에서 상태를 관리할 수 있도록 AppState의 StudySession에서 관리합니다.
  • 원활한 QA를 위해서 기본시간을 1분으로 설정했습니다.
  • 기존의 Modal 사용방식을 변경 했습니다.(TabView가 상위계층에 존재하는 문제)

+ TimePicker 관련 로직
TimePicker 관련 로직이 나름 복잡해서 해당 로직을 어디에 둘지 고민중입니다. 현재는 View에 로직이 존재하는데 조금만 더 생각을 해보고 적절한 곳으로 로직을 분리하도록 하겠습니다.

SCREENSHOT (Optional)

@Seokki-Kwon Seokki-Kwon added the 🔥 Feat 새로운 기능 관련 label Oct 27, 2024
@Seokki-Kwon Seokki-Kwon requested a review from owllwo98 October 27, 2024 19:36
@Seokki-Kwon Seokki-Kwon self-assigned this Oct 27, 2024
@Seokki-Kwon Seokki-Kwon merged commit 257bb22 into develop Oct 28, 2024
@Seokki-Kwon Seokki-Kwon deleted the Feature/#42-Study branch November 22, 2024 07:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔥 Feat 새로운 기능 관련
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant