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

[Refactor] 모임일정과 관련된 모듈을 재구성하고 리팩토링 합니다. #216

Merged
merged 15 commits into from
Dec 11, 2024

Conversation

Seokki-Kwon
Copy link

Related Issue

Description

  • Tuist 관점에서 기존에는 Feature 기준으로 모듈이 정확히 나눠져있지 않았습니다.
  • 모임일정 생성시 TCA모듈(Reducer)을 기능별로 재구성하여 관심사를 분리하고 복잡한 상태관리를 개선하고자 했습니다.
    • 기존의 단일 리듀서로 관리하던 모임 일정 기능을 모임 정보, 장소, 초대 친구 리듀서로 분리하고 RootReducer에서 개별 리듀서를 조합하여 사용하는 방식으로 변경
    • 장소 검색 기능또한 장소 검색, 카카오맵 별도의 리듀서로 분리하고 장소 검색이 카카오맵 리듀서를 embed 하는 방식으로 변경
  • 리팩토링을 진행하는 도중에 변경된 디자인과 기능에 대응합니다(친구 초대)

Screenshot (Optional)

Requirements for Review

친구 초대 화면은 별도의 flow로 다시 기획하신다고 하십니다(한개의 화면에 너무 많은 기능 및 개발시 복잡성 문제)

@Seokki-Kwon Seokki-Kwon added 🔧Fix 버그 수정 이슈 🔄Refactor 코드 리팩토링 이슈 labels Nov 27, 2024
@Seokki-Kwon Seokki-Kwon self-assigned this Nov 27, 2024
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.

와우..고생많으셨습니다..!!

@Seokki-Kwon Seokki-Kwon merged commit 5b35ba7 into develop Dec 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔧Fix 버그 수정 이슈 🔄Refactor 코드 리팩토링 이슈
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants