Feature/tgyuu/#69 #73

tgyuuAn · 2023-12-29T05:56:32Z

1. 📄 관련된 이슈 및 소개

closed #69

2. 🔥변경된 점

-DatePickerDialog 커스텀

-TimePickerDialog 커스텀

-일정 등록 로직에 Validation 기능 추가

-설문 페이지 UI 개선 (마진 값, 정렬 등)

-설문 등록 UI 개선 (마진 값, 정렬, Card Radius 등)

-일정 DB에 eventId Docoument ID로 랜덤생성되는 String 값으로 변경

-시작 날짜, 종료 날짜를 받도록 변경

3. 📸 스크린샷(선택)

-일정, 시간 설정 Dialog 커스텀

- 일정 등록 로직에 Validation 기능 추가

- 설문 조사, 설문 등록 페이지 UI 개선

피그마에 있는 UI랑 마진 값이나 Card Radius 안 맞는 부분이 맞아서 조금 손봤습니둥.

-일정 DB에 eventId Docoument ID로 랜덤생성되는 String 값으로 변경

이거 떔에 eventID Int였던게 String으로 바꿔져서 대규모 작업 좀 했습니다.

SurveyForm이나 Survey Entity에 있던 eventID도 다 갱신 해주었습니다.

-시작 날짜, 종료 날짜를 받도록 변경

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

- Compose에서 SnackBar 뛰우는 방법

https://kotlinworld.com/206

jeongjaino

고생했어요 태규상 !!

저는 비행기 기다리면서 살짝 쿵 리뷰 남겨봤습니다 하하

앞으로 열심히 와피 으라차차입니다. ~!

jeongjaino · 2024-01-02T08:19:22Z

core/data/src/main/java/com/wap/wapp/core/data/repository/event/EventRepositoryImpl.kt

+            title = eventTitle,
+            content = eventContent,
+            location = eventLocation,
+            startDateTime = eventStartDateTime.format(DateTimeFormatter.ISO_LOCAL_DATE_TIME),


해당 로직으로, data 모듈에서도 3번 포맷팅 사용되어서

network처럼 유틸로 빼버리는게 어떨까요 ?

DateUtil 로 빼놓겠습니다!

jeongjaino · 2024-01-02T08:22:18Z

core/network/src/main/java/com/wap/wapp/core/network/source/event/EventDataSourceImpl.kt

            firebaseFirestore.collection(EVENT_COLLECTION)
-                .document(getMonth(date))
+                .document(getMonth(startDateTime.toISOLocalDateTime().toLocalDate()))


해당 코드 괄호때문에 알아보기 쉽지 않은데, startDateTime ~ toLocalDate 까지 변수로 빼버리는 건 어때요 ?

아주 좋은 생각입니다 진호상. 수정해놓을게요!

jeongjaino · 2024-01-02T08:23:07Z

core/network/src/main/java/com/wap/wapp/core/network/source/survey/SurveyDataSource.kt

+    suspend fun isSubmittedSurvey(
+        eventId: String,
+        userId: String,
+    ): Result<Boolean>


이전부터 개행을 계속 하시던데,

따로 개행하시는 이유가 있을까요??

@jeongjaino

전 개행 무조건 해야하는 건 줄 알았어요........... 컨벤션 읽어보고 오겠습니다 ㅋㅋㅋㅋㅋㅋ큐ㅠ

꼭 안해도 되고, 줄바꿈이 필요한 경우에만 줄바꿈을 하는군요................

다시 원래대로 수정해놨습니다!

�하하 태규상 헷갈리게 제가 적어놨네요 하하하하

근데 오히려 개행하면, 매개변수 인식은 더 잘되서 오히려 굿입니다 하하

jeongjaino · 2024-01-02T08:24:57Z

...rc/main/java/com/wap/wapp/feature/management/registration/component/RegistrationTextField.kt

 import com.wap.designsystem.WappTheme

 @Composable
 internal fun RegistrationTextField(
    value: String,
    onValueChange: (String) -> Unit,
    modifier: Modifier = Modifier,
+    textAlign: TextAlign? = null,


따로 NullableType으로 지정하신 이유가 있을까요 ??

@jeongjaino

고 이유는 바로 TextStyle의 TextAlign의 기본 값이 null이기 때문입니다!

아무것도 넣어주지 않을 경우 일반 기본값으로 들어가게 했어요...!

jeongjaino · 2024-01-02T08:27:47Z

...src/main/java/com/wap/wapp/feature/management/registration/component/WappDatePickerDialog.kt

+}
+
+@Composable
+private fun DayOfWeek(modifier: Modifier = Modifier) {


일반적인 DayOfWeek에 컴포넌트 네이밍도 들어가면 좋을 것 같아요 !

컴포넌트 네이밍이라 하면 어떤 것이 있을까요..?

사실 이거 네이밍 하기 너무 애매해서 ㅠㅠㅠㅠㅠㅠㅠㅠㅠㅠㅠㅠㅠㅠㅠㅠ 추천 받습니다,...

GPT 선생님의 도움을 받아 CalendarWeekDays로 명명했습니다!

jeongjaino · 2024-01-02T08:31:19Z

...src/main/java/com/wap/wapp/feature/management/registration/component/WappDatePickerDialog.kt

+}
+
+@Composable
+private fun CalendarMonthItem(


태규상 해당 로직 일부 주석 처리 가능한가요 ??

로직이 많고 복잡한 부분은 주석이 있으면 이해하기 훨씬 더 좋을꺼 같아요

조오옿습니다. 달력 부분 일부 주석 처리 해놓을게요!

jeongjaino · 2024-01-02T08:33:00Z

...src/main/java/com/wap/wapp/feature/management/registration/component/WappTimePickerDialog.kt

+                        periodSelectorUnselectedContentColor = WappTheme.colors.gray95,
+                        periodSelectorSelectedContainerColor = WappTheme.colors.yellow34,
+                        periodSelectorUnselectedContainerColor = WappTheme.colors.black25,
+                    ),


ㄷㄷㄷㄷㄷㄷㄷㄷㄷㄷ 컬러 다 찍어보시면서 구현해보셨겠네요

고생하셨어요 !

제 노력을 알아주시니 감개무량할 따름입니다.

jeongjaino · 2024-01-02T08:34:19Z

...src/main/java/com/wap/wapp/feature/management/registration/event/EventRegistrationContent.kt

-                modifier = Modifier.padding(top = 30.dp),
+            DeadlineCard(
+                title = stringResource(R.string.start_date),
+                hint = startDate.format(DateTimeFormatter.ofPattern("yyyy.MM.dd")),


이거 따로 유틸에 구현되어 있지 않나요 ??

DateTimeFormatter.ofPattern("yyyy.MM.dd")

헉 맞습니다. 진호상 코드를 복붙했던 것 같은데......

다 수정해놓겠씁니다!

jeongjaino · 2024-01-02T08:35:08Z

...src/main/java/com/wap/wapp/feature/management/registration/event/EventRegistrationContent.kt

-                modifier = Modifier.fillMaxWidth(),
+            DeadlineCard(
+                title = stringResource(R.string.end_time),
+                hint = endTime.format(DateTimeFormatter.ofPattern("HH.mm")),


해당도 유사하게 커먼 모듈에 구현하시면 될 것 같아요 !

완료했씁니다!

jeongjaino · 2024-01-02T08:36:30Z

...c/main/java/com/wap/wapp/feature/management/registration/event/EventRegistrationViewModel.kt

+                eventStartTime = _eventStartTime.value,
+                eventEndDate = _eventEndDate.value,
+                eventEndTime = _eventEndTime.value,
+            ).fold(


반환 값을 따로 안받고 있어서, fold를 사용하시는 것 보다

onSuccess, onFailure 사용하시는 건 어때요 ??

@jeongjaino

사실 이 fold랑 onSuccess, onFailure 따로 쓰는 것의 차이점을 잘 모르고있었는데요,

한 번 찾아보겠씁니다.

fold

출처

간단한 분기처리만 처리해줄 때에는 onSuccess, onFailure를 사용하는군요!

굿입니다. 수정해놓을게요..!

으라차차차차초차촤촤촤

tgyuuAn added 16 commits December 28, 2023 23:57

[FEATURE] #69 : TimePicker 커스텀

2e8b297

[CHORE] #69 : TimePickerDialog 분리

19ea0a1

[FEATURE] #69 : DatePickerDialog 캘린더 이식

a6307fa

[FEATURE] #69 : DatePickerDialog 캘린더 UI 보완

c90a384

[FEATURE] #69 : 설문조사 등록 UI 보완

f6d90d0

[FEATURE] #69 : 관리 페이지 UI 보완

b0806c4

[FEATURE] #69 : WappDatePickerDialog 완성

7a7e3b8

[FEATURE] #69 : DeadLineCard 재사용 할 수 있도록 분리

41fe35a

[FEATURE] #69 : 설문조사 등록 UI 보완

7462c96

[FEATURE] #69 : 일정 등록 로직 대공사

3dd2a56

[FEATURE] #69 : 일정 추가 Dialog 적용 및 UI 개선

e556a44

[REFACTOR] #69 : eventId Int -> String 으로 변경

55682ba

[REFACTOR] #69 : RegistrationTextField 상단 부터 입력 받도록 수정

5dc38dc

[REFACTOR] #69 : 일정 등록 로직 변경

a0d381b

[FEATURE] #69 : 등록 SharedFlow 추가

533d391

[FEATURE] #69 : 일정 등록 Validation 추가

4b666eb

tgyuuAn requested a review from jeongjaino December 29, 2023 05:56

tgyuuAn self-assigned this Dec 29, 2023

tgyuuAn added 🌱기능🌱 새로운 기능 두두둥장! 🎨유아이🎨 피그마, 엑세멜, 컴포즈 유아이/유엑스 작업 🧩태규🧩 ENFP 안태규 24세 🔥리뷰 기다리는 중🔥 PR을 올리고 코드리뷰를 기다리고 있는 상태입니다. labels Dec 29, 2023

tgyuuAn added 7 commits January 1, 2024 22:47

[FEATURE] #69 : 마감 설정을 위해 StateFlow 추가

2cae0bf

[FEATURE] #69 : 시작 시간, 시작 날짜 파라미터 및 string 추가

22f257b

[FEATURE] #69 : 시작 시간, 시작 날짜 선택 기능 추가

288da2a

[FEATURE] #69 : 시작, 종료 다이얼로그 구분

d45b4ce

[FEATURE] #69 : DB Entity 시작 날짜 반영

92241e1

[FEATURE] #69 : EventResponse도 StartDateTime, EndDateTime으로 변경

ce1e79c

[FEATURE] #69 : 일정 데이터 디자인에 맞게 화면에 보여주도록 변경

ba26ef5

jeongjaino approved these changes Jan 2, 2024

View reviewed changes

tgyuuAn added 8 commits January 2, 2024 17:49

[REFACTOR] #69 : 너무 긴 구문 변수로 빼서 구분하기 쉽도록 변경

6af73f3

[REFACTOR] #69 : 달력 부분 약간의 주석 추가

5163eaa

[REFACTOR] #69 : DateTimeFormatter 미리 Util에 만들어진 Formatter 활용

f285487

[REFACTOR] #69 : 불필요한 Result fold 제거

a11d247

[REFACTOR] #69 : 불필요한 개행 제거

7e6b4bc

[REFACTOR] #69 : Kotlin 단일 표현식 코틀린 컨벤션 적용 안된 부분 적용

3f87779

[REFACTOR] #69 : LocalDateTime -> String Uitl로 분리

fc17542

[REFACTOR] #69 : DayOfWeek -> CalendarWeekDays

f027b29

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

tgyuuAn merged commit f95fb41 into develop Jan 3, 2024

tgyuuAn deleted the feature/tgyuu/#69 branch February 6, 2024 08:19

tgyuuAn mentioned this pull request Mar 12, 2024

#219/feature/jsp/backend pknu-wap/MediLenz#222

Open

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Feature/tgyuu/#69 #73

Feature/tgyuu/#69 #73

tgyuuAn commented Dec 29, 2023 •

edited

Loading

jeongjaino left a comment

jeongjaino Jan 2, 2024

tgyuuAn Jan 2, 2024 •

edited

Loading

jeongjaino Jan 2, 2024

tgyuuAn Jan 2, 2024

jeongjaino Jan 2, 2024

tgyuuAn Jan 2, 2024

tgyuuAn Jan 2, 2024 •

edited

Loading

jeongjaino Jan 3, 2024

jeongjaino Jan 2, 2024

tgyuuAn Jan 2, 2024 •

edited

Loading

jeongjaino Jan 2, 2024

tgyuuAn Jan 2, 2024

tgyuuAn Jan 2, 2024

jeongjaino Jan 2, 2024

tgyuuAn Jan 2, 2024

jeongjaino Jan 2, 2024

tgyuuAn Jan 2, 2024

jeongjaino Jan 2, 2024

tgyuuAn Jan 2, 2024

jeongjaino Jan 2, 2024

tgyuuAn Jan 2, 2024

jeongjaino Jan 2, 2024

tgyuuAn Jan 2, 2024

tgyuuAn Jan 2, 2024 •

edited

Loading

jeongjaino Jan 3, 2024

Feature/tgyuu/#69 #73

Feature/tgyuu/#69 #73

Conversation

tgyuuAn commented Dec 29, 2023 • edited Loading

1. 📄 관련된 이슈 및 소개

2. 🔥변경된 점

3. 📸 스크린샷(선택)

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

jeongjaino left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

tgyuuAn Jan 2, 2024 • edited Loading

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 Jan 2, 2024 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

tgyuuAn Jan 2, 2024 • edited Loading

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

tgyuuAn Jan 2, 2024 • edited Loading

Choose a reason for hiding this comment

fold

Choose a reason for hiding this comment

tgyuuAn commented Dec 29, 2023 •

edited

Loading

tgyuuAn Jan 2, 2024 •

edited

Loading

tgyuuAn Jan 2, 2024 •

edited

Loading

tgyuuAn Jan 2, 2024 •

edited

Loading

tgyuuAn Jan 2, 2024 •

edited

Loading