[fix] 3차 스프린트 QA 수정사항 반영 #248

HAJIEUN02 · 2024-03-10T14:49:31Z

Related issue 🛠

closed [fix] 3차 스프린트 QA 수정사항 반영 #244

Work Description ✏️

설명 텍스트와 인디케이터가 겹쳐있는 이슈 수정
'나의 단체' -> '입장 방식 선택' -> ’나의 단체’ 뷰로 복귀 시 종료 토스트 메시지 노출되는 버그 수정
나의 단체 뷰 : 단체 변경시 모달에서 단체명 13자 이상일 시에 12자까지만 노출시키고 ‘…’처리
더보기 뷰 단체명 2줄인 경우 고려해 수정
나의 단체 뷰 owner 표시 있는 경우 흰색 테두리 보이는 것 수정
신규 단체 개설 완료 안내 뷰 단체명 13자 이상일 경우 12자까지만 노출시키고 '...' 처리
기존 단체 입장 완료 안내 뷰 단체명 13자 이상일 경우 12자까지만 노출시키고 '...' 처리

Screenshot 📸

adf6bca9-bdf4-4488-9d72-cf0eefa6ea44.mp4

Uncompleted Tasks 😅

N/A

To Reviewers 📢

하나는 실수로 지현언니 브랜치에 해서 PR 링크 연결해두었습니다..ㅜ
죄송핑
-> 확장함수로 바꿔서 이제 여기서도 확인할 수 있음둥

영상은 처음 가입하고 [입장 방식 선택 뷰]에 접근했을 때 이전 버튼을 누르면 앱이 종료되고, 가입 이후 [나의 단체 뷰]를 통해 접근했을 때에는 이전 버튼을 누르면 나의 단체 뷰로 다시 돌아가는 플로우를 찍어서 좀 길어요ㅜ

…메시지 노출되는 버그 수정

DoReMinWoo

실기기 확인했는데 안짤리고 잘 나오네요!

DoReMinWoo · 2024-03-10T15:00:46Z

app/src/main/java/org/sopt/pingle/presentation/ui/onboarding/onboarding/OnboardingActivity.kt

+    private fun initLayout() {
+        when (intent.getStringExtra(FROM_ACTIVITY)) {
+            MY_GROUP_ACTIVITY -> Unit
+            else -> setDoubleBackPressToExit(binding.root)
+        }


오홍 코드 깔끔하네요

증말용? ㅎ.ㅎ 이걸 initLayout 함수 내에 넣는게 맞나 살짝 고민됐는데 어케 생각하세염 ㅜ

jihyunniiii

굿뜨

jihyunniiii · 2024-03-10T15:08:02Z

app/src/main/res/layout/activity_onboarding_explanation.xml

            android:layout_width="match_parent"
-            android:layout_height="match_parent"
-            app:layout_constraintTop_toBottomOf="@id/tv_onboarding_explanation_skip"
-            app:layout_constraintBottom_toTopOf="@id/tl_onboarding_indicator" />
+            android:layout_height="wrap_content"
+            app:layout_constraintTop_toBottomOf="@id/tv_onboarding_explanation_skip" />


width는 matchparent보다 0dp를 사용하는 게 좋아보이네요 !

넹 일어나자마자 수정할게여!

jihyunniiii · 2024-03-10T15:12:53Z

app/src/main/res/layout/activity_onboarding_explanation.xml

+            app:layout_constraintTop_toBottomOf="@id/vp_onboarding_explanation"
            app:layout_constraintStart_toStartOf="parent"
            app:layout_constraintBottom_toTopOf="@id/btn_onboarding_explanation_next"
-            android:layout_marginBottom="58dp"
+            android:layout_marginBottom="18dp"


이 탭 레이아웃 정렬과 관련해서 디자인 측이랑 대화를 한 번 해보시는 게 좋을 것 같아요.
지금 이 제약이라면 뷰페이저와 버튼 사이 공간에 탭 레이아웃이 가운데 정렬되게 되는데요.
디자인 측에서 원하는 디자인이 그게 맞는지,, 확인이 필요할 것 같아용 ~

이거 그리고 왜 18로 수정되는지 설명 부탁ㅇㅣ요 ~

이게 위 텍스트랑은 40dp 차이가 나고 아래 버튼이랑은 58dp 차이가 나는데요.. 위 아래 제약을 다 걸어서 중앙정렬이 된 상태에서 아래에 마진 18dp 추가하는 느낌으로 구현햇숨다

아아 이거 아래 마진 18 두고 중앙 정렬인지,, 아니면 그냥 버튼 위로 58만큼 떨어지게를 원하는 건지,, 아니면 뷰페이저 아래로 40을 원하는 건지 한 번 체크해줘요 ~~

jihyunniiii · 2024-03-10T15:16:56Z

app/src/main/java/org/sopt/pingle/presentation/ui/onboarding/onboarding/OnboardingActivity.kt

+        when (intent.getStringExtra(FROM_ACTIVITY)) {
+            MY_GROUP_ACTIVITY -> Unit
+            else -> setDoubleBackPressToExit(binding.root)
+        }


지렸핑 ㄷ.ㄷ when문 사용하신 건 추후 확장성을 고려한 건가요?

사실 확장성 고려라기보다는 if else문보다 훨씬 보기 깔끔하다고 느껴져서 이렇게 했는데 나중에 뷰가 추가된다거나 플로우가 좀더 정리되면 수정하기 편하겠군요
그러나!!!! const val은 확장성 고려해서 여러번 선언 안하구 두 뷰에서 import해서 활용했삼요 ㅎ.ㅎ 칭찬받고싶다

역시 잘한다 하지은 ㅠㅠ 진짜 짱 많이 늘었서 ㅜ 내가 다 뿌듯함

jihyunniiii · 2024-03-10T15:18:04Z

app/src/main/java/org/sopt/pingle/presentation/ui/mygroup/MyGroupActivity.kt

@@ -28,6 +30,7 @@ class MyGroupActivity : BindingActivity<ActivityMyGroupBinding>(R.layout.activit

    private val viewModel by viewModels<MyGroupViewModel>()
    private lateinit var adapter: MyGroupAdapter
+    private lateinit var resultLauncher: ActivityResultLauncher<Intent>


얜 없어도 될 것 같은ㄷㅔ요?

헉스 원래 이 방법을 쓰려다가 putExtra로 바꿔서 ,, ㅎ 역시 꼼꼼하다 지울게여ㅜ

jihyunniiii · 2024-03-11T23:10:41Z

디잔핑이 새 에셋 만들어주면 그것만 반영하고 머지하면 될 것 같네용 ~

jihyunniiii

고생핑

HAJIEUN02 added 3 commits March 10, 2024 21:51

[fix] #244 온보딩 추가설명 뷰 : 설명 텍스트와 인디케이터가 겹쳐져 있는 이슈 해결

80efd7d

[fix] #244 나의 단체 뷰 : '나의 단체' -> '입장 방식 선택' -> ’나의 단체’ 뷰로 복귀 시 종료 토스트 …

123c4ef

…메시지 노출되는 버그 수정

[chore] #244 ktlintFormat 적용

0641f39

HAJIEUN02 added 🛠️ fix 🛠️ 🐹 지은 🐹 labels Mar 10, 2024

HAJIEUN02 added this to the 3차 스프린트 QA milestone Mar 10, 2024

HAJIEUN02 self-assigned this Mar 10, 2024

HAJIEUN02 requested a review from a team as a code owner March 10, 2024 14:49

Merge branch 'develop' into fix-3rd-sprint-qa-jieun

7b38cf2

DoReMinWoo approved these changes Mar 10, 2024

View reviewed changes

[fix] #244 더보기 뷰 : 현재 단체명 2줄인 경우를 고려해 수정

e4f48cb

jihyunniiii approved these changes Mar 10, 2024

View reviewed changes

HAJIEUN02 added 4 commits March 11, 2024 22:25

[fix] #244 온보딩 추가설명 뷰 : 인디케이터가 움직이는 이슈 해결

df08dfe

[del] #244 온보딩 추가설명 뷰 : 필요없는 변수 삭제

054c784

[chore] #244 온보딩 추가설명 뷰 : match_parent를 0dp로 수정

e1d3841

[chore] #244 ktlintFormat 적용

ffb89ba

HAJIEUN02 added 6 commits March 12, 2024 09:34

[fix] #244 나의 단체 뷰 : owner 표시를 테두리가 꽉 찬 에셋으로 변경

c983e64

[feat] #244 단체명 13자 이상일 시 12까지만 노출시키고 ... 처리하는 확장함수 구현

68b04c4

[fix] #244 나의 단체 뷰 : 단체 변경 모달에 makeEllipsisGroupName 적용

c24d45a

[fix] #244 신규 단체 개설 완료 뷰 : 단체명에 makeEllipsisGroupName 적용

8a6a298

[fix] #244 기존 단체 입장 완료 뷰 : 단체명에 makeEllipsisGroupName 적용

8eb5406

[chore] #244 ktlintFormat 적용

009a320

jihyunniiii approved these changes Mar 12, 2024

View reviewed changes

HAJIEUN02 merged commit 26c22c9 into develop Mar 12, 2024
1 check passed

jihyunniiii deleted the fix-3rd-sprint-qa-jieun branch March 12, 2024 08:49

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[fix] 3차 스프린트 QA 수정사항 반영 #248

[fix] 3차 스프린트 QA 수정사항 반영 #248

HAJIEUN02 commented Mar 10, 2024 •

edited

Loading

DoReMinWoo left a comment

DoReMinWoo Mar 10, 2024

HAJIEUN02 Mar 10, 2024

jihyunniiii left a comment

jihyunniiii Mar 10, 2024

HAJIEUN02 Mar 10, 2024

jihyunniiii Mar 10, 2024

HAJIEUN02 Mar 10, 2024

jihyunniiii Mar 10, 2024

jihyunniiii Mar 10, 2024

HAJIEUN02 Mar 10, 2024

jihyunniiii Mar 10, 2024

jihyunniiii Mar 10, 2024

HAJIEUN02 Mar 10, 2024

jihyunniiii commented Mar 11, 2024

jihyunniiii left a comment

[fix] 3차 스프린트 QA 수정사항 반영 #248

[fix] 3차 스프린트 QA 수정사항 반영 #248

Conversation

HAJIEUN02 commented Mar 10, 2024 • edited Loading

Related issue 🛠

Work Description ✏️

Screenshot 📸

Uncompleted Tasks 😅

To Reviewers 📢

DoReMinWoo left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

jihyunniiii 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

jihyunniiii commented Mar 11, 2024

jihyunniiii left a comment

Choose a reason for hiding this comment

HAJIEUN02 commented Mar 10, 2024 •

edited

Loading