[BSVR-223/FEAT] 시야 후기 등록 TAB 이동 및 SnackBar 추가 #107

minju1459 · 2024-08-21T05:59:45Z

closed [FEATURE] 시야 후기 등록 좌석 선택 TAB 이동 및 SnackBar 추가 #104

💻주요 작업 내용

열만 입력, 번만 입력 (*휠체어석) -> DTO 수정
서버 변경 -> 익사이팅석, 프리미엄석 하드코딩
좌석선택- [구역선택]-[좌석번호] 간 탭이동
리뷰 등록 관련 스낵바 추가

🎞리뷰 요청 사항

이제 예외 케이스 및 열만 입력 등록 서버 배포돼서 잘됩니다. 확인부탁드려요 😎
제 뷰는 사소한 변경사항들이 많아, 큰 등록 플로우 변경되기 전에 먼저 구현하는게 맞다고 판단해서 이번에 pr 단위를 좀 작게 가져갑니다 !
2차 MVP 1 / 2 / 3 차 구현 중 두번째 pr입니다 ~! 아자자 🤪

Jokwanhee

고생했어~ 민주야~ 🐿️🐿️🐿️🐿️

Jokwanhee · 2024-08-21T06:42:35Z

core/src/main/java/com/dpm/core/base/BindingBottomSheetDialog.kt

@@ -13,7 +13,7 @@ abstract class BindingBottomSheetDialog<B : ViewBinding>(
    private val bindingInflater: (LayoutInflater, ViewGroup?, Boolean) -> B,
 ) : BottomSheetDialogFragment() {

-    private var _binding: B? = null
+    var _binding: B? = null


이거 왜 private 제거한거야?

Jokwanhee · 2024-08-21T06:54:10Z

presentation/src/main/java/com/dpm/presentation/seatreview/dialog/SelectSeatDialog.kt

+    override fun onDestroyView() {
+        super.onDestroyView()
+        _binding = null
+    }


요 함수 Base 추상 클래스에서 바인딩 해제해주고 있지 않나? 중복 코드같아~!

아이고 이거 왜 여기있어 ..? private 제거된 이유도 이거 때문이네 복구해놓을게 ~

Jokwanhee · 2024-08-21T06:58:53Z

presentation/src/main/java/com/dpm/presentation/seatreview/dialog/SelectSeatDialog.kt

@@ -524,4 +622,23 @@ class SelectSeatDialog : BindingBottomSheetDialog<FragmentSelectSeatBottomSheetB
            }
        }
    }
+
+    private fun makeSpotImageAppbar(message: String, marginHorizontal: Int) {
+        val parentView = requireDialog().window?.decorView?.findViewById<View>(android.R.id.content)


decorView를 사용한 이유가 있을까?

토스트 띄우는데 View 객체 넘겨주려면 아래 코드로 사용해도 좋을 것 같아

val parentView = binding.root.rootView

Jokwanhee · 2024-08-21T07:00:50Z

민주야 이게 두 번째 PR 이면 다왔는데,
이제 남은 태스크가 뭔지 알 수 있을까~? 👏

BENDENG1

확실히 자르면서 가니깐 리뷰하기도 더 좋아보이긴하네~!
민주 고생했어! 👍👍

BENDENG1 · 2024-08-21T06:47:26Z

presentation/src/main/java/com/dpm/presentation/seatreview/ReviewActivity.kt

+            messageColor = com.depromeet.designsystem.R.color.color_foreground_white,
+            icon = com.depromeet.designsystem.R.drawable.ic_alert_circle,
+            iconColor = com.depromeet.designsystem.R.color.color_error_secondary,
+            marginHorizontal = marginHorizontal,


엇 혹시 marginBottom아니야..?!

지금 qa 받구 있는데 추후에는 marginhorizontal을 삭제할 예정이야..!
(이미지 스낵바 같은경우 text 너비에 따른 wrap_content로 요구를 해서)

SpotImageSnackBar는 추후에 균욱 오빠가 수정해준다고 해서 안건드리는 선에서 스낵바 text 너비에 따라 marginhorizontal dp를 설정해줬어 그럼 일단 marginHorizontal 파라미터 삭제해놓을겡 ~~

BENDENG1 · 2024-08-21T06:50:28Z

presentation/src/main/java/com/dpm/presentation/seatreview/dialog/ReviewMySeatDialog.kt

+                icon = com.depromeet.designsystem.R.drawable.ic_alert_circle,
+                iconColor = com.depromeet.designsystem.R.color.color_error_secondary,
+                marginBottom = 96,
+                marginHorizontal = 26,


이부분도
93번 브랜치 이미지 스낵바
참고하면 좋을것 같아! 지금 아직 QA중이라😅

아 그리구 messageColor, icon, iconColor가 이제 모든 뷰에서 동일한 디자인 시스템이 적용되는데 아예 파라미터 값을 안받게 설정하는게 좋으려나?

난 동의 ! 지금 디자인 시스템에도 느낌표 아이콘 이미지가 있는 스낵바, endMessage가 있는 스낵바 이렇게 두개로만 통일되어 있는 것 같아서 동일하게 적용하면 좋을 것 같아 👍👍

BENDENG1 · 2024-08-21T06:51:52Z

presentation/src/main/java/com/dpm/presentation/seatreview/dialog/SelectSeatDialog.kt

-                tvNextBtn.visibility = VISIBLE
-                tvSelectZone.setTextColor(binding.root.context.colorOf(com.depromeet.designsystem.R.color.color_foreground_heading))
-                tvSelectNumber.setTextColor(binding.root.context.colorOf(com.depromeet.designsystem.R.color.color_foreground_caption))
+            llTabSelectSeat.setOnSingleClickListener {


이 부분 진짜 노고가 느껴진다..

BENDENG1 · 2024-08-21T07:00:32Z

core/src/main/java/com/dpm/core/base/BindingBottomSheetDialog.kt

-    private var _binding: B? = null
+    var _binding: B? = null


minju1459 · 2024-08-21T07:14:18Z

민주야 이게 두 번째 PR 이면 다왔는데, 이제 남은 태스크가 뭔지 알 수 있을까~? 👏

이제 좌석 시야 등록 / 직관 후기 등록 플로우 추가 태스크만 완료되면 될 것 같아 ~! 🐒

SsongSik

수고했엉~~~~이제 그만잘해~~~~

minju1459 added 10 commits August 19, 2024 23:33

[feat/#104] 좌석선택- [구역선택]-[좌석번호] 간 탭이동 구현

1bf53ff

[feat/#104] 구장 추가, 구역 먼저 선택 스낵바 추가

163a632

[feat/#104] 휠체어석 열 수정 불가 스낵바 추가

cba6e29

[fix/#104] 서버 변경 -> 익사이팅석, 프리미엄석 하드코딩

f8130e0

[fix/#104] 구역 미선택 -> 좌석 선택 시 모든 editText, 스피너 입력 막기

3d44718

[feat/#104] 유리막이 있는 자리 키워드 추가

5b8f8bd

[feat/#104] 후기 키워드는 각각 3개까지 선택할 수 있어요 스낵바 추가

d94ad8e

[feat/#104] 좌석 선택, 내 시야 후기 입력 상태에 따른 스낵바 및 업로드 버튼 UI 변경

e5e3e96

[feat/#104] 열만 입력, (휠체어석) 번호만 입력 추가로 인한 DTO 수정

0380462

[fix/#104] HTTP 403 에러 원인 파악 후 팀버 제거

4db0003

minju1459 added 🎇Feature 기능 개발 🦔개발자_박민주 박민주 labels Aug 21, 2024

minju1459 added this to the 2차 스프린트 개발 milestone Aug 21, 2024

minju1459 requested review from BENDENG1, SsongSik and Jokwanhee August 21, 2024 05:59

minju1459 self-assigned this Aug 21, 2024

minju1459 changed the title ~~[BSVR-223/FEAT] 시야 후기 등록 2차 - TAB 이동 및 SnackBar 추가~~ [BSVR-223/FEAT] 시야 후기 등록 TAB 이동 및 SnackBar 추가 Aug 21, 2024

Jokwanhee reviewed Aug 21, 2024

View reviewed changes

BENDENG1 reviewed Aug 21, 2024

View reviewed changes

minju1459 added 3 commits August 21, 2024 16:15

[fix/#104] decorView 제거

21958ac

[fix/#104] Base 추상 클래스 바인딩 해제 중복 코드

a6bfdd6

[fix/#104] 스낵바 추후 변경 사항 미리 적용

35680d9

minju1459 requested review from BENDENG1 and Jokwanhee August 21, 2024 07:26

SsongSik approved these changes Aug 21, 2024

View reviewed changes

minju1459 merged commit b188823 into main Aug 21, 2024
1 check passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[BSVR-223/FEAT] 시야 후기 등록 TAB 이동 및 SnackBar 추가 #107

[BSVR-223/FEAT] 시야 후기 등록 TAB 이동 및 SnackBar 추가 #107

minju1459 commented Aug 21, 2024

Jokwanhee left a comment

Jokwanhee Aug 21, 2024

Jokwanhee Aug 21, 2024

minju1459 Aug 21, 2024

Jokwanhee Aug 21, 2024

Jokwanhee commented Aug 21, 2024

BENDENG1 left a comment

BENDENG1 Aug 21, 2024

BENDENG1 Aug 21, 2024

minju1459 Aug 21, 2024 •

edited

Loading

BENDENG1 Aug 21, 2024

BENDENG1 Aug 21, 2024

minju1459 Aug 21, 2024 •

edited

Loading

BENDENG1 Aug 21, 2024

BENDENG1 Aug 21, 2024

minju1459 commented Aug 21, 2024

SsongSik left a comment

[BSVR-223/FEAT] 시야 후기 등록 TAB 이동 및 SnackBar 추가 #107

[BSVR-223/FEAT] 시야 후기 등록 TAB 이동 및 SnackBar 추가 #107

Conversation

minju1459 commented Aug 21, 2024

💻주요 작업 내용

🎞리뷰 요청 사항

Jokwanhee 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

Jokwanhee commented Aug 21, 2024

BENDENG1 left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

minju1459 Aug 21, 2024 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

minju1459 Aug 21, 2024 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

minju1459 commented Aug 21, 2024

SsongSik left a comment

Choose a reason for hiding this comment

minju1459 Aug 21, 2024 •

edited

Loading

minju1459 Aug 21, 2024 •

edited

Loading