[feat] 지역 인기 매장 API 연동 #17

Doreminwoo · 2023-11-27T16:05:42Z

Work Description ✏️

지역 인기 매장 API 연동

Screenshot 📸

Uncompleted Tasks 😅

이미지 모서리 둥글게 처리가 첫번째 아이템만 됌...

To Reviewers 📢

더미데이터 삭제

jihyunniiii · 2023-11-28T12:22:33Z

머지 전에 요건 필수적으로 고쳐주세용

TextView에 textColor 지정해주기 ! -> 지금 몇 개 빠진 게 있는 것 같은데 전체적으로 추가해주세용 !
리사이클러뷰 width 0dp로 지정해주깅 !!!
대기 n팀 나타내는 칩 패딩 설정 빠져있는 것 같아오 확인해주깅 !!!
item_popular_store에서 순위 나타내는 부분 textAppearance 설정 안 되어있어용 (글구 여기 마진 top을 주기보다는 이미지에 대해 가운데 정렬을 하는 게 좋을 것 같네요)
item_popular_store 정보 나타내는 부분 전체적으로 layout 묶은 다음 이미지에 대해 가운데 정렬 되도록 제약 설정해주세용

뷰 짤 때 꼼꼼하게 본다고 봤는데,,, 빠뜨린 부분이 많네요,,, 제송하지만 이것들 반영해주면 감사하겠숩니다,,,

jihyunniiii

서버통신 야무지다 !
잘했다 ! 체고다 !
제가 더 알려드릴게 없네용 ㅎ.ㅎ
나중에 셈나 때 코루틴 배우면 그거 적용해서 서버통신 함 해보세용 !
UI 적으로 수정해야할 부분만 수정하구 머지합시당 수고하셨어용
이미지 이슈는 제가 함 해보구 알려드릴게용 >.<
민우오빠 짱

jihyunniiii · 2023-11-28T12:32:02Z

app/src/main/java/org/sopt/tabling/presentation/store/PopularStoreActivity.kt

+        storeViewModel.popularStoreList.observe(this) { storeList ->
+            popularStoreAdapter.setStoreList(storeList)
+        }


셈나도 안 왔으면서 옵저버 패턴 벌써 잘 하는 거 진짜 체고야 ㅠ

jihyunniiii · 2023-11-28T12:32:28Z

app/src/main/java/org/sopt/tabling/presentation/store/StoreViewModel.kt

+            response.onSuccess { data ->
+                _popularStoreList.value = data.storeData.sortedBy { it.shopId }
+            }.onFailure {
+                Timber.tag("API").d("error")


Timber 누가 알려줬서 ㅠ 미쳐따

ㅋㅎㅋㅎ Log 쓰니깐 친절하게 Timber 쓰라고 알려주더라구요

Timber 사실 tag 없어두 되지롱 ㅋㅋ 저는 tag 없이 그냥 쓰는 편입니당

jihyunniiii · 2023-11-28T12:32:42Z

app/src/main/java/org/sopt/tabling/presentation/store/StoreViewModel.kt

+        viewModelScope.launch {
+            val response = runCatching { ServicePool.popularStoreService.getPopularStore() }
+            response.onSuccess { data ->
+                _popularStoreList.value = data.storeData.sortedBy { it.shopId }


정렬까지 야무지네용

jihyunniiii · 2023-11-28T12:33:00Z

app/src/main/res/drawable/shape_gray_200_6.xml

60인데 6이라 되어있어용

맞아요 확인하느라 그렇게 했었어요 ㅠㅠ
다시 반영했습니다!

jihyunniiii · 2023-11-28T12:33:41Z

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

-            android:layout_width="0dp"
-            android:layout_height="wrap_content"
-            android:layout_marginStart="16dp"
+            android:layout_width="wrap_content"


엥 이거 내가 이렇게 바꾸라 했구나,,,, 졸림 이슈로 잘못 말했서요 ㅠ 얘도 0dp로 해주셍요 ㅠㅠㅠㅠㅠ 제송해오

jihyunniiii · 2023-11-28T12:45:11Z

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

        <ImageView
            android:id="@+id/iv_popular_store"
            android:layout_width="0dp"
            android:layout_height="100dp"
-            android:layout_marginStart="12dp"
+            android:layout_marginStart="4dp"
            android:layout_marginTop="8dp"
            android:layout_marginBottom="8dp"
+            android:background="@drawable/shape_gray_200_6"
            android:scaleType="centerCrop"
            app:image="@{storeData.profilePhotoUrl}"
            app:layout_constraintBottom_toBottomOf="parent"


왜 이미지 라운드 처리 안 먹히는지 딱 알았숩니당 ㅋ
android:clipToOutline="true" 속성을 넣어보십쇼 !
요 속성을 적용해야 이미지가 백그라운드에 맞춰 잘리게 됩니당
그리구 지금 background가 이상한 게 들어가 있는 것 같아용
radius가 6인 백그라운드가 들어가야하는데 60짜리가 들어가 있서용
수정 부탁해용 ~

옹 그러네요!!

Doreminwoo · 2023-11-28T13:43:16Z

머지 전에 요건 필수적으로 고쳐주세용

TextView에 textColor 지정해주기 ! -> 지금 몇 개 빠진 게 있는 것 같은데 전체적으로 추가해주세용 !

리사이클러뷰 width 0dp로 지정해주깅 !!!

대기 n팀 나타내는 칩 패딩 설정 빠져있는 것 같아오 확인해주깅 !!!

item_popular_store에서 순위 나타내는 부분 textAppearance 설정 안 되어있어용 (글구 여기 마진 top을 주기보다는 이미지에 대해 가운데 정렬을 하는 게 좋을 것 같네요)

item_popular_store 정보 나타내는 부분 전체적으로 layout 묶은 다음 이미지에 대해 가운데 정렬 되도록 제약 설정해주세용

뷰 짤 때 꼼꼼하게 본다고 봤는데,,, 빠뜨린 부분이 많네요,,, 제송하지만 이것들 반영해주면 감사하겠숩니다,,,

5번은 어떻게 해야할지 모르겠어서 나머지만 커밋 푸시 했씁니다!

# Conflicts: # app/src/main/AndroidManifest.xml

HAJIEUN02

저번 코리 내 거 하느라 바빠서 제대로 못 읽어봐서 죄삼다ㅜㅜ 오늘 건 열심히 읽어봐쏘!!! 근데 양이 적어서 코리 달 게 많이 없네요ㅜㅜ 뷰 구현한 거 나중에 꼭 다시 읽어볼게 오빤 또 얼마나 야물딱진 코드를 짰는지~~ 고생했습니다💘

HAJIEUN02 · 2023-11-28T14:03:50Z

app/src/main/java/org/sopt/tabling/presentation/store/PopularStoreActivity.kt

@@ -27,11 +26,8 @@ class PopularStoreActivity :
    }

    private fun setPopularStoreList() {
-        storeViewModel.popularStoreList.observe(
-            this,
-            Observer { storeList ->


옵저버 패턴 이런 식으루 쓰는 거구남,, 세미나 듣고 오빠 코드로 복습하구 ~~ 좋으네요

HAJIEUN02 · 2023-11-28T14:05:09Z

app/src/main/java/org/sopt/tabling/presentation/store/StoreViewModel.kt

+        viewModelScope.launch {
+            val response = runCatching { ServicePool.popularStoreService.getPopularStore() }
+            response.onSuccess { data ->
+                _popularStoreList.value = data.storeData.sortedBy { it.shopId }


오와!!!! 정렬 어케 할지 고민했는데 이런 식으루 하면 되는구나
난 이용예정 이용완료 스트링으로 오는데 어케 정렬하지

자.. 자... 자동정렬을 애용합시다!
이 정렬이 아니고 sorted였군요 ㅋㅋㅋㅋㅋㅋ
구글과 챗지피티에 검색하면 다 나옵니다!

jihyunniiii · 2023-11-28T14:07:58Z

진짜 찐막 수정사항 입니다,,,

item_popular_store에서 순위 나타내는 부분 textAppearance 설정, textColor 설정 부탁해용
item_popular_store 이미지 부분에 테두리가 있었네요,,,? 지금 확인함 이슈,, 사용한 shape_gray_200_6 파일에서 solid 색상을 없애주고, item의 이미지에 foreground 속성을 줍시다.
가운데 정렬 어떻게 하는지는 카톡으로 보내줄게염 !

HAJIEUN02 · 2023-11-28T17:53:06Z

app/src/main/java/org/sopt/tabling/data/service/PopularStoreService.kt

+
+interface PopularStoreService {
+    @GET("/shops")
+    suspend fun getPopularStore(): ResponsePopularStoreDto


요기 왜 Response 형태가 아닌가요?? 그래두 상관없는건가여,,? 세미나 때 배운 내용은 꼭 Response<> 안에 담던데ㅜ,ㅜ

Call을 하기 위해서 사용하는건데 저는 enqueue를 사용하지 않고 코루틴으로 해보니깐 Call이 필요가 없더라구요!

jihyunniiii · 2023-11-28T23:58:14Z

shape 지정하는 것만 수정해줘용 !
데바를 진짜 야무지게 써서 잘 구현하셨네용
수고하셨습니당
저거 수정 마무리되면 머지해줘용 !

backgroun shape 반영

Doreminwoo added 5 commits November 28, 2023 00:58

[feat] #11 서버통신 구현

c3bb64d

더미데이터 삭제

[mod] #11 RecyclerView 맨 마지막 아이템 짤리는 현상 수정

cd1949f

[mod] #11 디자인 변경으로 인한 Store Id TextView 변경

80ff6fc

[add] #11 이미지 round 처리

6fff3aa

Merge branch 'develop' into feat-popular-store-api

cec51a9

Doreminwoo added 💫 feat 💫 ➡️ 민우 ➡️ labels Nov 27, 2023

Doreminwoo added this to the API 연동 milestone Nov 27, 2023

Doreminwoo requested review from HAJIEUN02 and jihyunniiii November 27, 2023 16:05

Doreminwoo self-assigned this Nov 27, 2023

Doreminwoo linked an issue Nov 27, 2023 that may be closed by this pull request

[feat] 지역 인기 매장 서버통신 구현 #11

Closed

7 tasks

Doreminwoo added 2 commits November 28, 2023 01:16

[chore] #11 ktlint 적용

c6d9ae9

[chore] #11 ktlint 적용

0a4d74a

jihyunniiii approved these changes Nov 28, 2023

View reviewed changes

jihyunniiii reviewed Nov 28, 2023

View reviewed changes

[chore] #11 코드리뷰 반영

ca2e2e2

Merge branch 'develop' into feat-popular-store-api

3c9b904

# Conflicts: # app/src/main/AndroidManifest.xml

HAJIEUN02 approved these changes Nov 28, 2023

View reviewed changes

Doreminwoo added 3 commits November 29, 2023 02:59

[chore] #11 코드리뷰 적용

667eee8

Merge branch 'develop' into feat-popular-store-api

4632e78

[chore] #11 ktlint 적용

113e1b0

[chore] #11 코드리뷰 반영

57ca887

backgroun shape 반영

Doreminwoo merged commit 1799eab into develop Nov 29, 2023
1 check passed

Doreminwoo deleted the feat-popular-store-api branch November 29, 2023 16:26

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[feat] 지역 인기 매장 API 연동 #17

[feat] 지역 인기 매장 API 연동 #17

Doreminwoo commented Nov 27, 2023 •

edited

Loading

jihyunniiii commented Nov 28, 2023

jihyunniiii left a comment

jihyunniiii Nov 28, 2023

jihyunniiii Nov 28, 2023

Doreminwoo Nov 28, 2023

jihyunniiii Nov 28, 2023

jihyunniiii Nov 28, 2023

jihyunniiii Nov 28, 2023

Doreminwoo Nov 28, 2023

jihyunniiii Nov 28, 2023

jihyunniiii Nov 28, 2023

Doreminwoo Nov 28, 2023

Doreminwoo commented Nov 28, 2023

HAJIEUN02 left a comment

HAJIEUN02 Nov 28, 2023

HAJIEUN02 Nov 28, 2023

Doreminwoo Nov 28, 2023 •

edited

Loading

jihyunniiii commented Nov 28, 2023

HAJIEUN02 Nov 28, 2023

Doreminwoo Nov 28, 2023

jihyunniiii commented Nov 28, 2023

[feat] 지역 인기 매장 API 연동 #17

[feat] 지역 인기 매장 API 연동 #17

Conversation

Doreminwoo commented Nov 27, 2023 • edited Loading

Work Description ✏️

Screenshot 📸

Uncompleted Tasks 😅

To Reviewers 📢

jihyunniiii commented Nov 28, 2023

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

Doreminwoo commented Nov 28, 2023

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

Doreminwoo Nov 28, 2023 • edited Loading

Choose a reason for hiding this comment

jihyunniiii commented Nov 28, 2023

Choose a reason for hiding this comment

Choose a reason for hiding this comment

jihyunniiii commented Nov 28, 2023

Doreminwoo commented Nov 27, 2023 •

edited

Loading

Doreminwoo Nov 28, 2023 •

edited

Loading