Fix/jaino/#81 #83

jeongjaino · 2024-01-05T08:34:28Z

1. 📄 관련된 이슈 및 소개

#81 마감기한이 지나도 설문 보이는 버그 수정

2. 🔥변경된 점

현재 마감기한이 남은 설문만 보이도록 수정
설문 화면에서 게스트인 경우 설문을 볼 수 없도록 수정

3. 📸 스크린샷(선택)

일반 비회원인 상황 (뒤로가기나, 디알로그 아웃사이드 누르면 디알로그 꺼지도록 구현)

(테스트가 따로 없어, 회원일 때 로그 찍고 / 운영진 인증 받고 다시 로그 찍기)

4. 💡알게된 혹은 궁금한 사항들

고민되는 상황
게스트 인 경우 디알로그로 표현하여 회원가입 유도
디알로그를 닫으면, 빈화면으로 표현

이럴꺼면, 그냥 설문에서 게스트 화면을 만드는게 나을까 고민이 되네요 하하

…그레이션

tgyuuAn

디스코드 화상채팅으로 로직 대부분 내용을 들어서 간단한 코멘트들만 달아봤어요!

너무 고생하셨습니다.

와피 다시 으라차차! 😽😽😽

tgyuuAn · 2024-01-05T14:11:33Z

core/data/src/main/java/com/wap/wapp/core/data/repository/management/ManagementRepository.kt

@@ -1,7 +1,7 @@
 package com.wap.wapp.core.data.repository.management

 interface ManagementRepository {
-    suspend fun getManager(userId: String): Result<Boolean>
+    suspend fun isManager(userId: String): Result<Boolean>


👍👍👍

tgyuuAn · 2024-01-05T14:12:11Z

core/domain/src/main/java/com/wap/wapp/core/domain/usecase/user/GetUserRoleUseCase.kt

+                        }
+                        UserRole.MEMBER
+                    },
+                    onFailure = { exception ->


이것도 단일 표현식이라서 코틀린 컨벤션 = 으로 치환할 수 있을 것 같아요...!

반영했습니다 !

tgyuuAn · 2024-01-05T14:13:05Z

core/model/src/main/java/com/wap/wapp/core/model/survey/SurveyForm.kt

+    }
+
+    fun isAfterDeadline(): Boolean {
+        val currentDateTime = LocalDateTime.now()


LocalDateTime.now보다 TimeZone이 들어간, DateUtil.generateNowDateTime()인가 ? 그거 쓰는게 어떨까요?!

좋습니다잇 !! 반영했습니다 !

tgyuuAn · 2024-01-05T14:14:04Z

feature/survey/src/main/java/com/wap/wapp/feature/survey/SurveyScreen.kt

+                        UserRole.MEMBER, UserRole.MANAGER -> { viewModel.getSurveyFormList() }
+                    }


와우 when 절에서 이렇게 , 를 사용해서 여러 개를 넣을 수도 있군요.

저는 처음 알았습니다...

tgyuuAn · 2024-01-05T14:14:36Z

feature/survey/src/main/java/com/wap/wapp/feature/survey/SurveyScreen.kt

@@ -141,7 +181,7 @@ private fun SurveyFormItemCard(
                    style = WappTheme.typography.titleBold,
                )
                Text(
-                    text = calculateDeadline(surveyForm.deadline),
+                    text = surveyForm.calculateDeadline(),
                    color = WappTheme.colors.yellow34,


👍👍👍

tgyuuAn · 2024-01-05T14:15:51Z

feature/survey/src/main/java/com/wap/wapp/feature/survey/SurveyViewModel.kt

+    sealed class UserRoleUiState {
+        data object Init : UserRoleUiState()
+        data class Success(val userRole: UserRole) : UserRoleUiState()
+    }
+
    sealed class SurveyFormListUiState {
        data object Init : SurveyFormListUiState()
        data class Success(val surveyFormList: List<SurveyForm>) : SurveyFormListUiState()


이게 제가 로직짤 때마다 항상 고민인데,

이러한 sealed class들을 ViewModel 안에 넣는게 맞을까요?

아니면 밖으로 빼는 것이 맞는걸까요 ...?

항상 고민이에요..

보면 제가 짠 로직들은 대부분 다 빼놨을거에요.

음 저도 처음에는 다 빼고 시작했는데, 여러 관리 측면 때문에 다 안으로 넣기 시작했어요

기본적으로 파일이 많아짐

State 네이밍이 겹치거나 비슷한 경우 실수할 여지가 있음.

viewModel.state로 접근해서 실수 여지를 줄여줌

저는 위의 경우 때문에 사용하긴 하는데,
따로 ViewModel의 코드 량이 많아지고, UiState를 ViewModel에서 접근해야 하는 불편함과 같은 단점도 있을 것 같아요

서로 장단점이 있어서 정답은 없지 않을까요 하하,,

이게 항상 고민이긴 해요.... 누가 명확한 답을 내려줬으면.....

확실히 진호님 말 들으니까 둘다 장단점이 있네요..

tgyuuAn · 2024-01-05T14:40:56Z

게스트 인 경우 디알로그로 표현하여 회원가입 유도

디알로그를 닫으면, 빈화면으로 표현

이럴꺼면, 그냥 설문에서 게스트 화면을 만드는게 나을까 고민이 되네요 하하

프로필 화면처럼 그냥 빈 화면을 표시해도 좋을 것 같아요.

다이얼로그가 닫히는 거라면 빈화면을 띄워줘야 하는데 좀 이상할 것 같기도 ..?

jeongjaino · 2024-01-05T15:03:03Z

게스트 인 경우 디알로그로 표현하여 회원가입 유도
디알로그를 닫으면, 빈화면으로 표현
이럴꺼면, 그냥 설문에서 게스트 화면을 만드는게 나을까 고민이 되네요 하하

프로필 화면처럼 그냥 빈 화면을 표시해도 좋을 것 같아요.

다이얼로그가 닫히는 거라면 빈화면을 띄워줘야 하는데 좀 이상할 것 같기도 ..?

빈화면은 아니구

설문

참가한 행사에 대해 설문에 응답하세요 !

는 남아있습니다 캬캬

tgyuuAn · 2024-01-05T15:08:33Z

@jeongjaino

근데 지금도 괜찮을 것 같기도 ...?

jeongjaino added 9 commits January 5, 2024 17:23

[FEATURE] #81 : 현재 날짜와 마감기한 비교하는 로직 구현 및 마감기한 자연어로 변환하는 로직 도메인 모델로 마이…

50f0585

…그레이션

[FIX] #81 : 현재 날짜와 비교하여, 마감기한이 후인 설문만 보이도록 수정

252ec60

[STYLE] #81 : Return Type Boolean 함수 네이밍 prefix get -> is로 수정

dc522bd

[FEATURE] #81 : 사용자 권한 도메인 모델 구현

b015e66

[FEATURE] #81 : 현재 사용자 권한 가져오는 유스케이스 구현

354a399

[FEATURE] #81 : 사용자 권한 상태를 가지는 Flow 및 권한 상태 Sealed Class 구현

ed2480b

[FEATURE] #81 : 설문 페이지에서 사용자가 게스트 인 경우 호출하는 디알로그 구현

1d2144a

[FEATURE] #81 : 설문 -> 로그인 화면 전환 로직 구현

124f76b

[CHORE] #81 : 설문 String Resource 추가

fc2bba2

jeongjaino added 🚀진호🚀 ESTP 정진호 23세 🌱기능🌱 새로운 기능 두두둥장! 🔥리뷰 기다리는 중🔥 PR을 올리고 코드리뷰를 기다리고 있는 상태입니다. 🔨픽스🔨 버그 및 잠재적 문제 해결 labels Jan 5, 2024

jeongjaino self-assigned this Jan 5, 2024

jeongjaino linked an issue Jan 5, 2024 that may be closed by this pull request

[FIX]: 마감기한이 지나도 설문 보이는 버그 수정 #81

Closed

3 tasks

tgyuuAn self-requested a review January 5, 2024 14:10

tgyuuAn reviewed Jan 5, 2024

View reviewed changes

tgyuuAn approved these changes Jan 5, 2024

View reviewed changes

tgyuuAn added 🌟머지 해주세요🌟 코드 리뷰가 완료된 뒤 PR을 올린사람이 Merge를 하면 되는 단계입니다. and removed 🔥리뷰 기다리는 중🔥 PR을 올리고 코드리뷰를 기다리고 있는 상태입니다. labels Jan 5, 2024

jeongjaino merged commit eb7037b into develop Jan 5, 2024
1 check passed

tgyuuAn deleted the fix/jaino/#81 branch February 6, 2024 08:20

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Fix/jaino/#81 #83

Fix/jaino/#81 #83

jeongjaino commented Jan 5, 2024 •

edited

Loading

tgyuuAn left a comment

tgyuuAn Jan 5, 2024

tgyuuAn Jan 5, 2024

jeongjaino Jan 5, 2024

tgyuuAn Jan 5, 2024

jeongjaino Jan 5, 2024

tgyuuAn Jan 5, 2024

tgyuuAn Jan 5, 2024

tgyuuAn Jan 5, 2024

jeongjaino Jan 5, 2024

tgyuuAn Jan 5, 2024

tgyuuAn commented Jan 5, 2024

jeongjaino commented Jan 5, 2024

tgyuuAn commented Jan 5, 2024

		UserRole.MEMBER, UserRole.MANAGER -> { viewModel.getSurveyFormList() }
		}

Fix/jaino/#81 #83

Fix/jaino/#81 #83

Conversation

jeongjaino commented Jan 5, 2024 • edited Loading

1. 📄 관련된 이슈 및 소개

2. 🔥변경된 점

3. 📸 스크린샷(선택)

4. 💡알게된 혹은 궁금한 사항들

tgyuuAn left a 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

tgyuuAn commented Jan 5, 2024

jeongjaino commented Jan 5, 2024

tgyuuAn commented Jan 5, 2024

jeongjaino commented Jan 5, 2024 •

edited

Loading