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

jihyunniiii · 2024-03-10T07:10:43Z

Related issue 🛠

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

Work Description ✏️

3차 스프린트 QA 수정사항을 반영하였습니다.
- 서치바 -> 입력 전 검색 버튼 비활성화
- 지도/리스트뷰 -> 검색창에서 뒤로가기 클릭 시 검색 취소 + 검색 페이지 이동
- 랭킹 -> 숫자 크기 12Bold -> 16Bold
- 랭킹 -> 데이터 텍소노미 스크롤 시작 시점으로 변경
- 단체 변경 후 홈 화면 진입 시 검색결과 유지되어 있는 문제 해결
- 검색 + 지도 뷰에서 칩 필터링을 진행하는 경우 검색 결과가 없어도 지도뷰 유지
- 리이슈 문제 완전 해결 ㅋ

Screenshot 📸

서치바 -> 입력 전 검색 버튼 비활성화

Screen_recording_20240310_103628.mp4

지도/리스트뷰 -> 검색창에서 뒤로가기 클릭 시 검색 취소 + 검색 페이지 이동

Screen_recording_20240310_160134.mp4

단체 변경 후 홈 화면 진입 시 검색결과 유지되어 있는 문제 해결

Screen_recording_20240311_144431.mp4

검색 + 지도 뷰에서 칩 필터링을 진행하는 경우 검색 결과가 없어도 지도뷰 유지

Screen_recording_20240311_215422.mp4

Uncompleted Tasks 😅

단체 변경 후 홈 화면 진입 시 검색결과 유지되어 있는 문제 해결
검색 + 지도 뷰에서 칩 필터링을 진행하는 경우 검색 결과가 없어도 지도뷰 유지

To Reviewers 📢

토큰 재발급.. 따로 이슈 파서 진행하겠습니다 ㅋㅋ 하암,,,
서치바 입력 전 검색 버튼 비활성화 -> 서치바가 있는 모든 뷰에 적용해달라고 하셔서 커스텀 뷰 만든 부분을 수정하였습니다. 요거 있는 뷰 담당한 분은 제외하고 해주시면 될 것 같아요 ~
검색 뷰 키보드 관련 디자인 측과 얘기해서 제외하기로 결정했습니다.
지도/리스트뷰 뒤로가기 관련 제 실기기에서 잘 되는 거 확인했는데,, 다른 분들도 한 번 확인해주시면 좋을 것 같습니다!
단체 변경 후 홈 화면 진입 시 검색결과 유지되어 있는 문제 해결 -> 이거 해결은 했는데,, 로직이 맘에 안 들어요,, 근데 SharedPreference를 사용하고 있는 한, 이 방법이 최선일 것 같기도 합니다,, (OnSharedPreferenceChangeListener를 사용해서 해결했어요)
검색 + 지도 뷰에서 칩 필터링을 진행하는 경우 검색 결과가 없어도 지도뷰 유지 -> ㅋㅋ 이전 검색 값을 저장하는 방식으로 구현하였는데,,, 로직이 맘에 안 들어요,, 좋은 방법 있다면 의견 플리즈
리이슈 문제 완전 해결 (인 듯함 ㅋㅋ)

…뜨도록 수정

DoReMinWoo

고고고고고고고고고고생핑~

DoReMinWoo · 2024-03-10T14:53:57Z

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

+        return if (groupName.length > GROUP_NAME_MAX_LENGTH) {
+            "${groupName.substring(SUBSTRING_START_INDEX, SUBSTRING_END_INDEX) + ELLIPSIS}"
+        } else {
+            groupName
+        }


여기 {} 없이 한줄로 깔끔하게 고고!?

@DoReMinWoo 제가 커밋한 부분이네욥 ㅋ.ㅋ 지현언니가 수정해준대요...

HAJIEUN02

빠르다빨라

HAJIEUN02 · 2024-03-10T15:00:33Z

app/src/main/java/org/sopt/pingle/presentation/ui/main/home/HomeFragment.kt

                override fun handleOnBackPressed() {
-                    homeViewModel.clearSearchWord()
-                    navigateToSearch()
+                    if (homeViewModel.searchWord.value.isNullOrEmpty()) {


이 부분 진짜 확장함수 쓰기 은근 어렵지 않나여?,, ㅜ.ㅜ 확장함수 그거 하나로 다 해결하고 싶은데 도저히 어떻게 해야할지 머르겠네요.. 온보딩 추가설명 뷰에서도 결국엔 실패했는데

아니 그니까용,,, 뒤로가기 버튼 관리 쉽지 않타,,, 이거 이번 스프린트 끝나고 어떻게 하면 잘 관리할 수 있을지 고민해 봐야겠서염 ㅠ

HAJIEUN02 · 2024-03-10T15:01:05Z

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

+        return if (groupName.length > GROUP_NAME_MAX_LENGTH) {
+            "${groupName.substring(SUBSTRING_START_INDEX, SUBSTRING_END_INDEX) + ELLIPSIS}"
+        } else {
+            groupName
+        }


@DoReMinWoo 제가 커밋한 부분이네욥 ㅋ.ㅋ 지현언니가 수정해준대요...

…ceChangeListener)

…flow로 통합

jihyunniiii · 2024-03-11T23:09:36Z

코리 달아준 이후로 수정한 부분이 많아서 코드 한 번만 다시 봐주시면 감사하겠슴둥

@DoReMinWoo @HAJIEUN02

HAJIEUN02

지도뷰는 진짜ㅜ.ㅜ 너무 어려워보이네요 늘 고생하는 젼언니 짱..

HAJIEUN02 · 2024-03-12T01:24:23Z

app/src/main/java/org/sopt/pingle/presentation/ui/main/home/HomeViewModel.kt

+        }
+
+    init {
+        localStorage.sharedPreference.registerOnSharedPreferenceChangeListener(


여기서 sharedPreference 내용 바뀌는 거 관찰하려고 localDataSource에 쉐프 그 자체를 추가한 거군용😶‍🌫️

HAJIEUN02 · 2024-03-12T01:24:47Z

app/src/main/java/org/sopt/pingle/presentation/ui/main/home/HomeViewModel.kt

-    private val _category = MutableStateFlow<CategoryType?>(null)
-    val category get() = _category.asStateFlow()
+    private val sharedPreferenceChangeListener =
+        SharedPreferences.OnSharedPreferenceChangeListener { _, key ->


진짜 OnSharedPreferenceChangeListener 이런 게 있는지는 상상도 못했네요,, 좋은 정보 감사합니당

HAJIEUN02 · 2024-03-12T01:26:41Z

app/src/main/java/org/sopt/pingle/presentation/ui/main/home/HomeViewModel.kt

    private val _mainListPingleListState = MutableSharedFlow<UiState<List<MainListPingleModel>>>()
    val mainListPingleListState get() = _mainListPingleListState.asSharedFlow()

-    fun setCategory(category: CategoryType?) {
-        _category.value = category
+    private fun clearPingleFiler() {


요기 Filter인데 Filer로 오타난 것 같아욤

HAJIEUN02 · 2024-03-12T01:27:09Z

app/src/main/java/org/sopt/pingle/presentation/ui/main/home/HomeViewModel.kt

-    val category get() = _category.asStateFlow()
+    private val sharedPreferenceChangeListener =
+        SharedPreferences.OnSharedPreferenceChangeListener { _, key ->
+            if (key == GROUP_ID) {


변화한 값(key)가 GROUP_ID인 경우에 클리어해주는 로직이군여!_! 근데 groupName도 같이 바뀌니까 GROUP_ID && GROUP_NAME 이런 식으로 둘다 처리해주면 어떨까요!? 그러면 좀더 안정적이지 않을까요

그럼 아마 if문 안으로 안 들어갈 거에요 ㅠ 이게 바뀔 때마다 수행되는 거라서 ㅜ

HAJIEUN02 · 2024-03-12T01:28:46Z

app/src/main/java/org/sopt/pingle/presentation/ui/main/home/HomeFragment.kt

+        homeViewModel.pingleFilter.flowWithLifecycle(viewLifecycleOwner.lifecycle)
+            .distinctUntilChanged()
+            .onEach { pingleFilter ->
+                Log.e("ㅋㅋ", pingleFilter.toString())


로그 지워주셔도 될 것 같아염ㅎ.ㅎ

와 싓 감삼다

HAJIEUN02 · 2024-03-12T01:30:31Z

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

@@ -142,15 +143,24 @@ class MyGroupActivity : BindingActivity<ActivityMyGroupBinding>(R.layout.activit
        }
    }

+    private fun makeEllipsisGroupName(groupName: String): String = when {


요 부분 when문으로 변경해주신거 반영해서 확장함수로 빼놨어요 제 피알에서!!

…ix-3rd-sprint-qa-jihyun

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

…ix-3rd-sprint-qa-jihyun

jihyunniiii added 4 commits March 10, 2024 10:10

[chore] #243 랭킹 아이템 숫자 크기 12Bold -> 16Bold

eae8e5d

[chore] #243 랭킹 데이터 텍소노미 스크롤 시작 시점으로 변경

a38eca9

[chore] #243 PingleSearch -> EditText가 비었을 시 검색 버튼 비활성화

1ed2ef1

[chore] #243 지도/리스트 뷰 -> 검색창에서 뒤로가기 클릭 시 검색 취소 + 검색페이지로 이동

f35c0b3

jihyunniiii added 🛠️ fix 🛠️ 🐸 지현 🐸 labels Mar 10, 2024

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

jihyunniiii self-assigned this Mar 10, 2024

jihyunniiii requested a review from a team as a code owner March 10, 2024 07:10

HAJIEUN02 and others added 4 commits March 10, 2024 22:18

[fix] #244 나의 단체 뷰 : 단체 변경시 모달에서 단체명 13자 이상일 시에 12자까지만 노출시키고 ‘…’처리

a9446a1

[chore] #244 나의 단체 뷰 : const val -> private const val로 수정

c4dbae0

[fix] #244 온보딩 추가설명 뷰 : 0개 단체 가입된 사용자 앱 재실행 시 온보딩 추가설명 뷰 입장 방식 선택 뷰가 …

78640d9

…뜨도록 수정

[chore] #243 ktlintFormat

4ba3b9c

DoReMinWoo approved these changes Mar 10, 2024

View reviewed changes

HAJIEUN02 approved these changes Mar 10, 2024

View reviewed changes

jihyunniiii added 8 commits March 11, 2024 12:03

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

e9fdc83

[chore] #243 makeEllipsisGroupName when 문으로 수정

2aebbd7

[chore] #243 PingleLocalDataSource에 sharedPreference 추가

0f0c84e

[chore] #243 단체 변경 후 홈 화면 진입 시 검색 결과가 유지되어 있는 문제 해결 (OnSharedPreferen…

1775e47

…ceChangeListener)

[chore] #243 검색 + 지도 뷰에서 칩 필터링을 진행하는 경우 검색 결과가 없어도 지도뷰 유지

0b48ceb

[feat] #243 홈 뷰 핑글 정보 조회를 위한 정보를 하나로 관리하기 위한 모델 생성

011a9b6

[feat] #243 홈 뷰 핑글 정보 조회를 위한 정보 (카테고리, 검색어, HomeViewType, Order) 하나의 …

54f60c0

…flow로 통합

[chore] #243 ktlintFormat

041e09a

HAJIEUN02 approved these changes Mar 12, 2024

View reviewed changes

HAJIEUN02 and others added 5 commits March 12, 2024 14:40

Merge remote-tracking branch 'origin/fix-3rd-sprint-qa-jihyun' into f…

ca2b255

…ix-3rd-sprint-qa-jihyun

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

5dbf5e8

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

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

8c87a8a

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

[chore] #243 로그아웃/탈퇴 후 이동 화면 수정 (Onboarding)

6471797

[chore] #243 Log 삭제

4e27cbf

jihyunniiii and others added 6 commits March 12, 2024 15:16

[chore] #243 오타 수정

1d28f7b

[chore] #243 로그아웃/탈퇴 후 이동 액티비티 수정 (auth)

22a28bc

[chore] #243 ktlintFormat

751d77f

Merge remote-tracking branch 'origin/fix-3rd-sprint-qa-jihyun' into f…

2cf81e5

…ix-3rd-sprint-qa-jihyun

[fix] #243 로그아웃/탈퇴 후 온보딩, 최초 접속 시 온보딩 뒤로가기 버튼 로직 분리

b95ce26

[chore] #243 ktlintFormat

993395f

jihyunniiii merged commit f6fb272 into develop Mar 12, 2024
1 check passed

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

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

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

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

jihyunniiii commented Mar 10, 2024 •

edited

Loading

DoReMinWoo left a comment

DoReMinWoo Mar 10, 2024

HAJIEUN02 Mar 10, 2024

HAJIEUN02 left a comment

HAJIEUN02 Mar 10, 2024

jihyunniiii Mar 11, 2024

HAJIEUN02 Mar 10, 2024

jihyunniiii commented Mar 11, 2024

HAJIEUN02 left a comment

HAJIEUN02 Mar 12, 2024

HAJIEUN02 Mar 12, 2024

HAJIEUN02 Mar 12, 2024

HAJIEUN02 Mar 12, 2024

jihyunniiii Mar 12, 2024

HAJIEUN02 Mar 12, 2024

jihyunniiii Mar 12, 2024

HAJIEUN02 Mar 12, 2024

jihyunniiii Mar 12, 2024

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

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

Conversation

jihyunniiii 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

HAJIEUN02 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

jihyunniiii commented Mar 11, 2024

HAJIEUN02 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

jihyunniiii commented Mar 10, 2024 •

edited

Loading