[mod] 상세페이지 홈페이지, 인스타, 전화번호 없는 경우에 대한 분기처리, 상세페이지, 검색뷰, 필터뷰 이슈 해결 #229

jihyunniiii · 2024-02-18T08:27:58Z

Related issue 🛠

closed [mod] 상세페이지 홈페이지, 인스타, 전화번호 없는 경우에 대한 분기처리, 상세페이지, 검색뷰, 필터뷰 이슈 해결 #226

Work Description ✏️

상세페이지 홈페이지, 인스타, 전화번호가 없는 경우에 대한 분기처리를 진행하였습니다.
검색 뷰 관련 오류를 해결하였습니다. (item_bakery에서 역이 완전히 표시되지 않는 오류, 검색 2번 진행 시 앱이 터지는 오류)
상세페이지 관련 오류를 해결하였습니다. (북마크 클릭 시 칩이 계속 쌓이는 오류)
필터뷰 건너뛰기 터치영역 조절

Screenshot 📸

검색 및 상세페이지 분기처리

Screen_recording_20240218_172516.mp4

북마크

Screen_recording_20240218_172213.mp4

Uncompleted Tasks 😅

N/A

To Reviewers 📢

지금 영상에는 전화번호 없는 경우에 빈 값이 뜨는데 이거 실제 더미 쌓을 때 - 로 넣어주신다고 하셨습니다. (홈페이지, 인스타가 없는 경우 -> " " 내려줌 + 뷰에서 사라지도록 구현, 전화번호 없는 경우 -> - 내려줌 + 뷰에 - 표시)
하다가 북마크 클릭했을 때 칩이 계속 쌓이는 오류를 발견해서 그것도 고쳤어요 ~ (사진 참고 ㅋ.ㅋ)

initBreadTypeChips 함수 확장함수로 빼는 게 나을지 고민하다가 그냥 놔뒀는데,, 뭐가 나은 것 같으신가요,,?

jooyyoo

수고하셨숨당🥖

jooyyoo · 2024-02-19T03:35:29Z

app/src/main/java/com/sopt/geonppang/presentation/myPage/MyBookMarksActivity.kt

-                        this.toBreadTypePointM2Chip(layoutInflater)
-                    }
-                )
+    private fun initBreadTypeChips(


흠 그러게요 chip 관련이라 같이 따로 확장함수로 빼도 괜찮을 것 같긴 합니당

이게 제가 모든 로직 확장함수로 빼려다가 일부만 뺀게, 리스트로 넘어오는 타입(BreadFilterType, ReviewData 등,, ) 이 다 다른데 거기서 breadType을 generic으로 받아서 접근할 수 가 없음여,,, 가능하면 빼면 좋을 거 같긴 하네염 근데 이게 경우의 수가 겁나 많아서 좀 고민이 필요함!

마자요 ㅜ 저도 그래서 빼려다가 안 뺐는데 ㅜ 이따가 같이 고민해보면 좋을 것 같습니다 ~~

Dan2dani

수고티비입니다

Dan2dani · 2024-02-19T03:59:18Z

app/src/main/java/com/sopt/geonppang/presentation/myPage/MyBookMarksActivity.kt

-                        this.toBreadTypePointM2Chip(layoutInflater)
-                    }
-                )
+    private fun initBreadTypeChips(


이게 제가 모든 로직 확장함수로 빼려다가 일부만 뺀게, 리스트로 넘어오는 타입(BreadFilterType, ReviewData 등,, ) 이 다 다른데 거기서 breadType을 generic으로 받아서 접근할 수 가 없음여,,, 가능하면 빼면 좋을 거 같긴 하네염 근데 이게 경우의 수가 겁나 많아서 좀 고민이 필요함!

Dan2dani · 2024-02-19T04:14:57Z

app/src/main/java/com/sopt/geonppang/data/model/response/ResponseDetailReview.kt

@@ -53,7 +53,7 @@ data class ResponseDetailReview(
                    memberNickname = review.memberNickname,
                    reviewId = review.reviewId,
                    reviewText = review.reviewText,
-                    recommendKeywordList = review.toRecommentKeyword()
+                    recommendKeywordList = review.toRecommendKeyword()


ㅋㅋㅋㅋㅋ 오 감사합니다

Dan2dani · 2024-02-19T04:15:19Z

app/src/main/java/com/sopt/geonppang/presentation/common/BakeryAdapter.kt

        ) {
            binding.bakery = bakery
            binding.executePendingBindings()

            // TODO: dana 다른 방식이 있는지 고민, 매 바인딩마다 removeAllViews 해야하는가 ?
            with(binding.cgBakeryBreadTypes) {
                this.removeAllViews()
-                initBreadTypeChips(this, position)
+                initBreadTypeChips(this, bakery.breadTypeList)


오옹 이 방식으로 햇어야 했네여

Dan2dani · 2024-02-19T04:16:29Z

app/src/main/java/com/sopt/geonppang/presentation/search/SearchActivity.kt

-                        this.toBreadTypePointM2Chip(layoutInflater)
-                    }
-                )
+    private fun initBreadTypeChips(


굳굳! 입니닷

나중에 이런 방식으로 다 바꾸는 것도 괜찮을 거 같네염!

음하하 ㅋ.ㅋ

Dan2dani · 2024-02-19T06:08:52Z

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

@@ -228,7 +228,7 @@

            <TextView
                android:id="@+id/tv_item_detail_bakery_info_homepage"
-                visibility="@{!bakeryInfo.homepageUrl.isEmpty()}"
+                visibility="@{!bakeryInfo.homepageUrl.trim().isEmpty()}"


trim()을 적용해야 하는 이유가 있나염?!

서버에서 "" 이게 아니라 " " (공백 있게) 이걸로 내려준대요 ㅠ

아항 ㅜㅜ

jihyunniiii added 6 commits February 18, 2024 16:03

[chore] #226 오타 수정

c5438bd

[feat] #226 홈페이지, 인스타가 없는 경우에 대한 분기처리 진행

e78cb12

[chore] #226 전화번호 레이아웃 수정

1823575

[fix] #226 item_bakery 역 표시 오류 해결

e8760e6

[fix] #226 검색 2번 진행 시 앱이 꺼지는 문제 해결

ab9b67a

[fix] #226 북마크 클릭 시 칩이 계속 쌓이는 문제 해결

ff5a888

jihyunniiii added 🐸 지현 🐸 🔮mod 🔮 labels Feb 18, 2024

jihyunniiii added this to the [릴리즈] 4차 스프린트 milestone Feb 18, 2024

jihyunniiii requested review from jooyyoo and Dan2dani February 18, 2024 08:27

jihyunniiii self-assigned this Feb 18, 2024

jooyyoo approved these changes Feb 19, 2024

View reviewed changes

jooyyoo reviewed Feb 19, 2024

View reviewed changes

Dan2dani approved these changes Feb 19, 2024

View reviewed changes

Dan2dani reviewed Feb 19, 2024

View reviewed changes

[fix] #226 필터뷰 건너뛰기 터치 이벤트 수정 (터치 영역이 이상하게 잡히는 문제 해결)

606d220

jihyunniiii merged commit 6495015 into develop Feb 19, 2024
1 check passed

jihyunniiii deleted the mod-detail-view branch February 19, 2024 15:45

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[mod] 상세페이지 홈페이지, 인스타, 전화번호 없는 경우에 대한 분기처리, 상세페이지, 검색뷰, 필터뷰 이슈 해결 #229

[mod] 상세페이지 홈페이지, 인스타, 전화번호 없는 경우에 대한 분기처리, 상세페이지, 검색뷰, 필터뷰 이슈 해결 #229

jihyunniiii commented Feb 18, 2024 •

edited

Loading

jooyyoo left a comment

jooyyoo Feb 19, 2024

Dan2dani Feb 19, 2024

jihyunniiii Feb 19, 2024

Dan2dani left a comment

Dan2dani Feb 19, 2024

Dan2dani Feb 19, 2024

Dan2dani Feb 19, 2024

Dan2dani Feb 19, 2024

Dan2dani Feb 19, 2024

jihyunniiii Feb 19, 2024

Dan2dani Feb 19, 2024

jihyunniiii Feb 19, 2024

Dan2dani Feb 19, 2024

[mod] 상세페이지 홈페이지, 인스타, 전화번호 없는 경우에 대한 분기처리, 상세페이지, 검색뷰, 필터뷰 이슈 해결 #229

[mod] 상세페이지 홈페이지, 인스타, 전화번호 없는 경우에 대한 분기처리, 상세페이지, 검색뷰, 필터뷰 이슈 해결 #229

Conversation

jihyunniiii commented Feb 18, 2024 • edited Loading

Related issue 🛠

Work Description ✏️

Screenshot 📸

Uncompleted Tasks 😅

To Reviewers 📢

jooyyoo 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

Dan2dani 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 Feb 18, 2024 •

edited

Loading