🔗 :: (#657) 신청화면 잔류신청하기 태그 업데이트 안됨 #689

parkuiery · 2024-06-07T06:05:15Z

개요

신청화면 태그 업데이트 로직을 추가했습니다

작업사항

추가 로 할 말

JunJaBoy · 2024-06-08T03:29:33Z

feature/src/main/java/team/aliens/dms/android/feature/main/application/ApplicationScreen.kt

+                    visible = appliedTitle != null,
+                    enter = slideInVertically() + fadeIn(),
+                    exit = slideOutVertically() + fadeOut(),
+                ) {
+                    if (appliedTitle != null) {


AnimatedVisibility에서 false면 밑에 if문이 의미 없는거 아닌가요?

혹시 어디가 false라고 말씀하시는 건가요?

코드 한 번 해석해보시죠

수정했습니다

JunJaBoy · 2024-06-08T03:30:30Z

feature/src/main/java/team/aliens/dms/android/feature/main/application/ApplicationViewModel.kt

+
+    override fun onCreate(owner: LifecycleOwner) {
+        super.onCreate(owner)
+        viewModelScope.launch {
+            fetchAppliedStudyRoom()
+            fetchAppliedRemainsOption()
+        }
    }


init 대신 oncreate를 넣으신 이유가 있나요? LIfeCycleOwner를 사용하는 것 같지 않은데..

init을 사용하면 신청을 하고 뒤로돌아가기를 했을 때 태그가 업데이트가 되지 않아 생명주기를 이용하여 onCreate 될 때마다 업데이트 하도록 만들었습니다.

음.. 이렇게 구현하는 것도 괜찮지만 신청 페이지에서 onDispose할 때 신청 생성/취소/변경 이벤트를 넘겨주고 이 페이지에서 기존 상태와 비교했을 때 변경이 있을때만 서버에서 값을 조회하는게 더 효율적일 것 같은데 어떻게 생각하세요?

좋은거 같습니다.
근데 이 기능을 빨리 사용해야 해서 우선 머지하고 나중에 리펙토링 하는쪽으로 해도 될까요?

좋습니다 😎
TODO 이슈 남겨주세요

JunJaBoy · 2024-06-10T10:58:18Z

feature/src/main/java/team/aliens/dms/android/feature/main/application/ApplicationScreen.kt

+                            vertical = PaddingDefaults.Small,
+                        ),
+                    ) {
+                        Text(text = appliedTitle!!)


이거 실행 해보셨나요? 터질 것 같은데

위 animatedvisibility가 visible이든 아니든(appliedTitle이 null이든 아니든) 이 밑에 블럭은 실행될 것 같아요

수정했습니다

JunJaBoy · 2024-06-10T10:59:32Z

feature/src/main/java/team/aliens/dms/android/feature/main/application/ApplicationViewModel.kt

+
+    override fun onCreate(owner: LifecycleOwner) {
+        super.onCreate(owner)
+        viewModelScope.launch {
+            fetchAppliedStudyRoom()
+            fetchAppliedRemainsOption()
+        }
    }


음.. 이렇게 구현하는 것도 괜찮지만 신청 페이지에서 onDispose할 때 신청 생성/취소/변경 이벤트를 넘겨주고 이 페이지에서 기존 상태와 비교했을 때 변경이 있을때만 서버에서 값을 조회하는게 더 효율적일 것 같은데 어떻게 생각하세요?

JunJaBoy · 2024-06-12T00:47:15Z

feature/src/main/java/team/aliens/dms/android/feature/main/application/ApplicationViewModel.kt

+
+    override fun onCreate(owner: LifecycleOwner) {
+        super.onCreate(owner)
+        viewModelScope.launch {
+            fetchAppliedStudyRoom()
+            fetchAppliedRemainsOption()
+        }
    }


좋습니다 😎
TODO 이슈 남겨주세요

JunJaBoy · 2024-06-12T00:47:53Z

feature/src/main/java/team/aliens/dms/android/feature/main/application/ApplicationScreen.kt

+                AnimatedVisibility(
+                    visible = appliedTitle != null,
+                    enter = slideInVertically() + fadeIn(),
+                    exit = slideOutVertically() + fadeOut(),
+                ) {
+                    if(appliedTitle!= null) {


흠 결국 원상복귀군요..

parkuiery added 2 commits June 7, 2024 15:03

♻️ :: 아이콘 변경

0278549

♻️ :: 태그 업데이트 로직 추가

bff97a8

parkuiery added refactor 코드 리팩토링 할 경우 👾bug 버그가 발생한 경우 labels Jun 7, 2024

parkuiery requested a review from JunJaBoy June 7, 2024 06:05

parkuiery self-assigned this Jun 7, 2024

parkuiery linked an issue Jun 7, 2024 that may be closed by this pull request

신청화면 잔류신청하기 태그 업데이트 안됨 #657

Closed

JunJaBoy requested changes Jun 8, 2024

View reviewed changes

parkuiery requested a review from JunJaBoy June 8, 2024 08:00

♻️ :: null 처리 삭제

ea412a7

JunJaBoy requested changes Jun 10, 2024

View reviewed changes

♻️ :: null 처리

5ed7c99

parkuiery requested a review from JunJaBoy June 11, 2024 06:32

JunJaBoy approved these changes Jun 12, 2024

View reviewed changes

♻️ :: todo

6f30327

parkuiery merged commit d30195c into develop Jun 12, 2024
1 check passed

parkuiery deleted the fix/657-신청화면-잔류신청하기-태그-업데이트-안됨 branch June 12, 2024 03:01

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

🔗 :: (#657) 신청화면 잔류신청하기 태그 업데이트 안됨 #689

🔗 :: (#657) 신청화면 잔류신청하기 태그 업데이트 안됨 #689

parkuiery commented Jun 7, 2024

JunJaBoy Jun 8, 2024

parkuiery Jun 8, 2024

JunJaBoy Jun 8, 2024

parkuiery Jun 10, 2024

JunJaBoy Jun 8, 2024

parkuiery Jun 8, 2024

JunJaBoy Jun 10, 2024

parkuiery Jun 11, 2024

JunJaBoy Jun 12, 2024

JunJaBoy Jun 10, 2024

JunJaBoy Jun 10, 2024

parkuiery Jun 10, 2024

JunJaBoy Jun 10, 2024

JunJaBoy Jun 12, 2024

JunJaBoy Jun 12, 2024

🔗 :: (#657) 신청화면 잔류신청하기 태그 업데이트 안됨 #689

🔗 :: (#657) 신청화면 잔류신청하기 태그 업데이트 안됨 #689

Conversation

parkuiery commented Jun 7, 2024

개요

작업사항

추가 로 할 말

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment