Feature/jaino/#128 #144

jeongjaino · 2024-02-27T12:04:45Z

1. 📄 관련된 이슈 및 소개

#128 설문 이전 버튼 클릭시 음답 검증 및 상태 저장

2. 🔥 변경된 점

설문 이전 버튼을 클릭하면 상태를 저장합니다.
설문 등록시 (운영진) 마지막 버튼 텍스트가 "다음"에서 "설문 등록"으로 변경되었습니다.
설문 작성시 뒤로가기 버튼이 close 버튼으로 변경되었습니다.

3. ✅ 꼭 확인해줬으면 하는 부분

준성님이 언급해주신 QA
회원이 설문 작성할 때, 설문 Validation이 통과되지 않았는데 뒤로가기 버튼이 수행된다.

를 수정하려고 했는데, 사실 제가 처음 개발할 때 그게 의도였어요.
뒤로가기에는 원래 Validation이 없어요. 그래서 응답 목록에도 Validation이 지켜지지 않은 채로 응답이 들어갈 수 있어요

하지만, 앞으로가기가 안돼요. 다음 버튼이나 제출버튼을 누르려면, Validation을 검증해야해요.
따라서 전체적으로 Validation이 지켜진 응답 목록을 가질 수 있어요

실제로 사용해보시면 사용자 측면에서 체감이 될꺼에요.
뒤로가기를 하고 싶은데, 현재 응답을 Validation이 수행되어야 뒤로 돌아가는 것과
형식이 지켜지지 않아도 뒤로가기가 수행되는 차이요.

근데 준성님 QA 덕분에 엄청 큰 문제를 발견했어요.
이전 버튼을 눌렀을 때, Validation은 수행이 안되어도 응답은 저장되어야하거든요
중요한 응답을 저장하는 로직이 빠졌었어요 !

그래서 이번 피알에 뒤로가기시 응답을 저장하는 로직을 추가했어요. (Validation을 검증하지 않아도)

제가 여러번 테스트하고, 생각했을 떄는 문제가 없었는데
태규상이 생각하시는 문제가 있으시면 답변주세요 ! (이해하셨을지 모르겠네요 하하,, =이해안되시면 디코 달려갈게요)

4. 📸 스크린샷(선택)

Screen_recording_20240227_210413.webm

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

…re/jaino/#95 # Conflicts: # app/src/main/java/com/wap/wapp/MainActivity.kt # feature/survey/src/main/java/com/wap/wapp/feature/survey/answer/SurveyAnswerForm.kt

…re/jaino/#95 # Conflicts: # app/src/main/java/com/wap/wapp/MainActivity.kt # feature/management/src/main/java/com/wap/wapp/feature/management/ManagementScreen.kt

…를 불러오는 게 아닌 인자로 넘긴 상태로 변경되도록 수정

… 현재 및 다음 상태 인자로 넘겨주기

…ontent

tgyuuAn

진호상 LGTM이에요~~

진호님이 올려놓으신 영상이 맞는 것 같긴하네요.

근데 한 가지 궁금한 게 있는데,

질문 추가 버튼을 누르면 무조건 질문을 만들어야 하나요?

되돌리기가 안되나요? 그게 그 #117 (comment) 이슈 인가...?!

tgyuuAn · 2024-02-27T12:11:04Z

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

+        val startDate = _eventStartDate.value
+        val endDate = _eventEndDate.value
+
+        if (startDate == endDate) {
+            return eventTime > _eventStartTime.value


정말 미세 먼지긴 한데,

eventStartTime만 변수로 안 빼고 value로 접근하니 약간 이질감(?)이 드는 느낌?!

ㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋ 태규상 제대로 보셨네요 저는 못알아차렸어요

수정완료했습니다잇!

jeongjaino · 2024-02-27T13:30:15Z

진호상 LGTM이에요~~

진호님이 올려놓으신 영상이 맞는 것 같긴하네요.

근데 한 가지 궁금한 게 있는데,

질문 추가 버튼을 누르면 무조건 질문을 만들어야 하나요?

되돌리기가 안되나요? 그게 그 #117 (comment) 이슈 인가...?!

네 태규상 지금 당장은 그렇습니다.

그래서 항목추가를 하면 무조건 질문을 만들어야 해요.

그래서 삭제 기능이 필요한거기도 하고요 😆

jeongjaino added 20 commits February 6, 2024 21:18

[CHORE] #95 : 다음/제출 버튼 컴포넌트 생성 -> 버튼 내에서 분기에 따라 텍스트만 변경되도록 수정

f2019aa

[CHORE] #95 : LaunchedEffect 안 viewModel.apply로 적용

2469613

Merge branch 'develop' of https://github.com/pknu-wap/WAPP into featu…

c19ab64

…re/jaino/#95 # Conflicts: # app/src/main/java/com/wap/wapp/MainActivity.kt # feature/survey/src/main/java/com/wap/wapp/feature/survey/answer/SurveyAnswerForm.kt

[CHORE] #95 : 병합 충돌 해결

1bfc1f4

Merge branch 'develop' of https://github.com/pknu-wap/WAPP into featu…

06a118d

…re/jaino/#95 # Conflicts: # app/src/main/java/com/wap/wapp/MainActivity.kt # feature/management/src/main/java/com/wap/wapp/feature/management/ManagementScreen.kt

[UI] #126 : 일정 등록 상세 페이지 날짜 컴포넌트와 버튼간의 간격 설정

bffbc80

[UI] #126 : 스크롤 컬럼 외부에 적용된 패딩 -> 안쪽의 패딩으로 변경

0938fa7

[UI] #126 : 일정 수정시 BottomBar Hide

00ef3ee

[FEATURE] #126 : EventRegistrationState 설정 ViewModel 함수 로직 수정 / 다음 상태…

98532a8

…를 불러오는 게 아닌 인자로 넘긴 상태로 변경되도록 수정

[FEATURE] #126 : onRegisterButton, onPreviousButton, onNextButton 클릭시…

8840a59

… 현재 및 다음 상태 인자로 넘겨주기

[FEATURE] #126 : 버튼클릭시 검증 후 다음 상태로 변환되도록 구현

8ad2658

[CHORE] #126 : String Resource 추가

3918d99

[MERGE] #126 : 병합 충돌 해결

783abfb

[UI] #126 : Event, SurveyForm Indicator와 Content 사이 40dp로 고정

d24de47

[UI] #126 : ProgressBar Horizontal Margin 설정

91ba85d

[CHORE] #126 : add String Resource SurveyFrom Registration Animated C…

37f21da

…ontent

[CHORE] #128 : 설문 등록 마지막 버튼 텍스트, 다음 -> 설문등록으로 변경

a238496

[CHORE] #128 : 설문 응답 Limit 8글자로 변경

5201149

[CHORE] #128 : 이전 버튼 클릭시 답변 수정되도록 구현

fde88b8

[CHORE] #128 : Formmating

6941989

jeongjaino added 🚀진호🚀 ESTP 정진호 23세 🔥리뷰 기다리는 중🔥 PR을 올리고 코드리뷰를 기다리고 있는 상태입니다. 🔨픽스🔨 버그 및 잠재적 문제 해결 labels Feb 27, 2024

jeongjaino self-assigned this Feb 27, 2024

tgyuuAn approved these changes Feb 27, 2024

View reviewed changes

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

tgyuuAn linked an issue Feb 27, 2024 that may be closed by this pull request

[FIX]: 설문응답 이전 버튼 클릭시, 응답 검증 및 상태 저장 버그 수정 #128

Closed

3 tasks

[CHORE] #128 : isValidEndTime, _eventStartTime 변수로 빼기

83a72f2

jeongjaino merged commit f2e53e3 into develop Feb 27, 2024

tgyuuAn deleted the feature/jaino/#128 branch March 8, 2024 05:35

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Feature/jaino/#128 #144

Feature/jaino/#128 #144

jeongjaino commented Feb 27, 2024

tgyuuAn left a comment

tgyuuAn Feb 27, 2024

jeongjaino Feb 27, 2024

jeongjaino Feb 27, 2024

jeongjaino commented Feb 27, 2024 •

edited

Loading

Feature/jaino/#128 #144

Feature/jaino/#128 #144

Conversation

jeongjaino commented Feb 27, 2024

1. 📄 관련된 이슈 및 소개

2. 🔥 변경된 점

3. ✅ 꼭 확인해줬으면 하는 부분

4. 📸 스크린샷(선택)

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

tgyuuAn left a comment

Choose a reason for hiding this comment

tgyuuAn Feb 27, 2024

Choose a reason for hiding this comment

jeongjaino Feb 27, 2024

Choose a reason for hiding this comment

jeongjaino Feb 27, 2024

Choose a reason for hiding this comment

jeongjaino commented Feb 27, 2024 • edited Loading

jeongjaino commented Feb 27, 2024 •

edited

Loading