[Feat/#145] taejung third qa #150

imtaejugkim · 2025-08-06T14:26:04Z

✅ 𝗖𝗵𝗲𝗰𝗸-𝗟𝗶𝘀𝘁

merge할 브랜치의 위치를 확인해 주세요(main❌/develop⭕)
리뷰가 필요한 경우 리뷰어를 지정해 주세요
리뷰는 (아직 미정)에 진행됩니다.
P1 단계의 리뷰는 (아직 미정)까지 반영합니다.
Approve된 PR은 assigner가 머지하고, 수정 요청이 온 경우 수정 후 다시 push를 합니다.

📌 𝗜𝘀𝘀𝘂𝗲𝘀

closed [Fix] 태정 3차 QA #145

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

투두 추가/수정/삭제 시 투두, 투데이 화면 반영
스케쥴 추가/수정/삭제 시 투데이 화면 반영
다이얼로그 지연 오류 수정

📷 𝗦𝗰𝗿𝗲𝗲𝗻𝘀𝗵𝗼𝘁

Screen_recording_20250806_232326.mp4

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

event bus 와 shared flow를 사용해 이벤트를 공유해보았습니다. 메모리 해제가 중요할 것 같은데, 관련해서 체킹이 좀 더 필요할 것 같습니다.
코리 만관부!

into feat/#145-taejung-third-qa

…도록 수정

hyoeunjoo

진짜 이게 되네. 이걸 해내네. 역시 김태정.
전 앞으로 시내버스 안타고 이벤트 버스 타고 다닐겁니다

hyoeunjoo · 2025-08-07T07:32:20Z

app/src/main/java/org/memento/core/event/EventBus.kt

+@Singleton
+class EventBus
+    @Inject
+    constructor() {
+        private val _events = MutableSharedFlow<EventType>()
+        val events = _events.asSharedFlow()
+
+        suspend fun emit(event: EventType) {
+            _events.emit(event)
+        }
+    }


앱 전역에서 이벤트를 발행하고 수신할 수 있도록 SharedFlow를 기반으로 EventBus를 잘 설계해주신 것 같습니다.
sealed class EventType로 이벤트 타입을 명확하게 관리한 덕에 타입 안정성까지;

답은 이벤트 버스였습니다.. 갓태정!

hyoeunjoo · 2025-08-07T07:39:54Z

app/src/main/java/org/memento/presentation/todo/TodoViewModel.kt

+            viewModelScope.launch {
+                _deleteState.value = UiState.Loading
+                val result = todoRepository.deleteTodo(toDoId = planId)
+
+                _deleteState.value =
+                    result.fold(
+                        onSuccess = {
+                            viewModelScope.launch {
+                                eventBus.emit(EventType.TodoDeleted)
+                            }
+                            UiState.Success(Unit)
+                        },
+                        onFailure = { throwable ->
+                            Timber.e(throwable, "Failed to delete Todo")
+                            UiState.Failure
+                        },
+                    )


여기 viewModelScope가 중첩되어 들어가있는데 혹시 의도가 있으셨던걸까요?
중첩 launch는 불필요한 코루틴 컨텍스트 생성을 유발할 수 있다구 알고 있어서 코드리뷰 남겨봅니다 ㅎㅎ

모든 코드에 같은 로직을 추가했더니, 추가하는 과정에 lauch도 그대로 추가를 했네요.. 수정하겠습니다!

hwidung

최고!! 오빠 먼저 머지하면 내꺼 그 위에 리팩해서 머지하면 될 것 같옹 수고했숑~~

hwidung · 2025-08-11T04:40:05Z

app/src/main/java/org/memento/core/event/EventBus.kt

이렇게 추상화 해주신 부분을 잘 활용하면 될 것 같습니다!! 좋은 설계 땡큐베리망치

hwidung · 2025-08-11T04:47:45Z

app/src/main/java/org/memento/presentation/plusbottomsheet/AddScheduleScreen.kt

+        focusManager.clearFocus(force = true)
+
+        val inputMethodManager = context.getSystemService(Context.INPUT_METHOD_SERVICE) as? InputMethodManager
+        inputMethodManager?.hideSoftInputFromWindow((context as? Activity)?.currentFocus?.windowToken, 0)


재사용 가능성이 있다면 따로 키보드 내리는 확장함수로 만들어도 될 것 같아요!

좋은 것 같아요. 반영하겠습니다!

hwidung · 2025-08-11T04:54:00Z

app/src/main/java/org/memento/presentation/today/TodayViewModel.kt

+                    viewModelScope.launch {
+                        eventBus.emit(EventType.TodoUpdated)
+                    }


위에 event bus를 잘 정의해준 덕분에 이렇게 편하게 쓸 수 있구나!! 나도 리팩할 때 타입 지정해서 이렇게 할겜!!

hwidung

저희의 큰 문제점이었던 렌더링 문제를 해결했지만!! 그런데 사실 Todo 에 변화가 생길 때마다 getTodoList을 호출하는 방식이니 어느 정도 불필요한 자원을 낭비하는 부분이 있다고 생각했어요

나중에 리팩토링 할 수 있다면 Room+Flow 사용하면 조금 더 효율적으로 해결할 수 있을 것 가타요

저희가 추가를 할 때마다 서버와 통신을 함과 동시에 Room db에도 저장하면, Room이 데이터 변경을 감지하고 관찰 중이던 Flow에 변경된 목록을 자동으로 emit 하면 이것도 방법이 될 수 있다! 그래서 이 부분도 저희가 계속 알아봤으면 좋겠다~ 란 생각이 들었어요오옹

imtaejugkim and others added 29 commits June 10, 2025 23:41

[fix/#143] 시간 역전 오류 endDate 추가 수정

b833f08

[fix/#143] 시간 역전 오류 calendar 로직으로 수정

11badf6

[fix/#143] schedule screen 출력 시 스크린 내려가게 수정

174d354

[refactor/#143] ktlint Format

98dd025

[fix/#145] 다이얼로그 뷰모델 제거 및 데이터 전달 수정

dee2971

[fix/#145] todayscreen detail 다이얼로그 관리 로직 수정

17af808

[fix/#145] todoscreen detail 다이얼로그 관련 데이터 변경

f8581fb

[refactor/#145] ktlint Format

f2db552

[fix/#145] 다이얼로그 todo 일정 완료 체크 시 todayscreen 반영 수정

d7bb40d

[fix/#145] todo screen 다이얼로그 적용 및 상세 화면 적용 수정

994a079

[feat/#145] today 데이터 반영에 따른 event type 및 bus 작성

521a664

[feat/#145] today 데이터 event bus 발생 시 데이터 갱신 작성

97878d9

[feat/#145] 일정 추가, 수정, 삭제 시 eventbus에 emit 하여 변경 알림

202f26c

[feat/#145] event bus 의 변경된 데이터를 바탕으로 갱신 코드 수정

1b418e1

[refactor/#145] ktlint Format

847de35

[refactor/#145] 자연어 처리 파싱 후 텍스트 trim 되던 오류 수정

25a9896

[refactor/#145] 데이터 갱신 eventbus 로직 반영, 바텀시트 불필요한 갱신 로직 삭제

f11075d

[refactor/#145] eventbus collect 활용 수정

f9e1d19

[refactor/#145] ktlint Format

9bc3299

[refactor/#145] todoviewmodel 데이터 업데이트 상태 추가 (일정 수정, 삭제, 추가)

77ccd27

[refactor/#145] ktlint Format

3c9fb31

[fix/#145] 사용하지 않는 다이얼로그 reset 함수 삭제

5c933c0

[refactor/#145] todoviewmodel 에서 schedule event 관리 제거

7f3dec4

[refactor/#145] ktlint Format

6373d0f

[fix/#145] todo screen schedule 로직 삭제 및 refreshtrigger 설정

d104885

Merge branch 'develop' of https://github.com/dev-memento/ANDROID-MEMENTO

f946c4f

into feat/#145-taejung-third-qa

[fix/#145] todo, today refreshtrigger flow를 replay=1 로 변경하여 최신 업데이트 되…

050bc02

…도록 수정

[fix/#145] time picker bottomsheet 드래그 안되도록 수정

ad07dde

[refactor/#145] ktlint Format

54c97ee

imtaejugkim requested a review from hwidung August 6, 2025 14:26

imtaejugkim requested a review from hyoeunjoo August 6, 2025 14:26

imtaejugkim self-assigned this Aug 6, 2025

imtaejugkim added 🤢태정🤢 ♻️ [REFACTOR] 리팩터링 (전면 수정) 🔧 [FIX] 버그 및 오류 해결 labels Aug 6, 2025

hyoeunjoo approved these changes Aug 7, 2025

View reviewed changes

hwidung approved these changes Aug 11, 2025

View reviewed changes

hwidung reviewed Aug 11, 2025

View reviewed changes

imtaejugkim added 4 commits August 16, 2025 14:15

[refactor/#145] 불필요한 중복 코루틴 스코프 생성 삭제

f8f6176

[refactor/#145] 키보드 및 포커스 관련 함수 추가

4b1f050

[refactor/#145] ktilnt Format

ab32db8

[refactor/#145] ktlint Format

ac7d94d

imtaejugkim merged commit d7054b6 into develop Aug 17, 2025
1 check passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

[Feat/#145] taejung third qa #150

[Feat/#145] taejung third qa #150

Uh oh!

imtaejugkim commented Aug 6, 2025

Uh oh!

hyoeunjoo left a comment

Uh oh!

hyoeunjoo Aug 7, 2025

Uh oh!

hyoeunjoo Aug 7, 2025

Uh oh!

imtaejugkim Aug 15, 2025

Uh oh!

hwidung left a comment

Uh oh!

hwidung Aug 11, 2025 •

edited

Loading

Uh oh!

hwidung Aug 11, 2025

Uh oh!

imtaejugkim Aug 15, 2025

Uh oh!

hwidung Aug 11, 2025

Uh oh!

hwidung left a comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

[Feat/#145] taejung third qa #150

[Feat/#145] taejung third qa #150

Uh oh!

Conversation

imtaejugkim commented Aug 6, 2025

✅ 𝗖𝗵𝗲𝗰𝗸-𝗟𝗶𝘀𝘁

📌 𝗜𝘀𝘀𝘂𝗲𝘀

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

📷 𝗦𝗰𝗿𝗲𝗲𝗻𝘀𝗵𝗼𝘁

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

Uh oh!

hyoeunjoo left a comment

Choose a reason for hiding this comment

Uh oh!

hyoeunjoo Aug 7, 2025

Choose a reason for hiding this comment

Uh oh!

hyoeunjoo Aug 7, 2025

Choose a reason for hiding this comment

Uh oh!

imtaejugkim Aug 15, 2025

Choose a reason for hiding this comment

Uh oh!

hwidung left a comment

Choose a reason for hiding this comment

Uh oh!

hwidung Aug 11, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

hwidung Aug 11, 2025

Choose a reason for hiding this comment

Uh oh!

imtaejugkim Aug 15, 2025

Choose a reason for hiding this comment

Uh oh!

hwidung Aug 11, 2025

Choose a reason for hiding this comment

Uh oh!

hwidung left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

hwidung Aug 11, 2025 •

edited

Loading