[feat] 나의 단체 뷰 구현 #201

HAJIEUN02 · 2024-02-24T15:11:10Z

Related issue 🛠

closed [feat] 나의 단체 뷰 구현 #194

Work Description ✏️

Screenshot 📸

XRecorder_01032024_145050.mp4

Uncompleted Tasks 😅

단체 변경 로직
모달 새로 만들기(detail이 없는 모달이 나의 단체 뷰에 있어서 마진 값 수정된 버전의 모달이 필요합니다!)
chip 머지되면 적용(현재 PingleBadge로 되어있음)

To Reviewers 📢

단체 변경 프로세스를 구현을 하긴 했는데 더 깔끔하게 바꾸고 싶어요.. 그러나
오늘이 뷰 마감이다보니...... 온보딩 코드리뷰부터 반영하러 갈게욥

Dan2dani

복사하기나 공유하기 누르고 나서 띄워져있는 뭐라고 해야하지 네모 박스 안사라져도 되나영?

Dan2dani · 2024-02-25T03:53:05Z

app/src/main/java/org/sopt/pingle/data/datasourceimpl/local/PingleLocalDataSourceImpl.kt

@@ -55,6 +55,26 @@ class PingleLocalDataSourceImpl @Inject constructor(
        get() = pref.getString(GROUP_NAME, "") ?: ""
        set(value) = pref.edit { putString(GROUP_NAME, value) }

+    override var meetingCount: String


오옹,, 이 정보 서버에서 아무것도 안내려주는건가요..? 어느 api에서도,,?

서버에서 내려주는 값이 있고 그거 사용할 수 있다면 너무 많은 정보를 로컬에 저장해놓는건 핸드폰 기종에 따라 약간 무리일수도 ~~

이거 단체 디테일 조회 API 있어서 그거 사용하면 될 듯 합니다 ~

api 연동하면서 로직도 수정하겠습니다! localStorage에는 정말 필요한 정보만 집어넣어두는 걸로 할게욥!!

Dan2dani · 2024-02-25T03:54:41Z

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

+
+    override fun getItemViewType(position: Int): Int {
+        return if (currentList[position].isOwner) {
+            2


상수화 하는게 좋겠네여 ~~

넹 반영해서 상수화 했숨다!!

jihyunniiii · 2024-02-25T05:37:14Z

선생님 이거 지금 빌드가 안 되는 것 같은데 확인 한 번만 부탁드려용 ! 일단 빌드 되게 바꿔놓고 코리 달고 있을게용

HAJIEUN02 · 2024-02-25T06:05:09Z

선생님 이거 지금 빌드가 안 되는 것 같은데 확인 한 번만 부탁드려용 ! 일단 빌드 되게 바꿔놓고 코리 달고 있을게용

어 마자요 그 모달 이미 만들어져 있는 거에 detail이 null이어야 작동해서 그렇게 수정해두고 커밋은 안 했거든욥
빌드 오류 나는게 마자용 ㅜ.ㅜ 띄우는거 테스트해보려고 코드만 작성해뒀어요
암튼 그래서 그 모달은 새로 만들 겁니당!!

# Conflicts: # app/src/main/res/values/strings.xml

jihyunniiii

수고하셨숩니당

jihyunniiii · 2024-02-25T05:39:12Z

app/src/main/java/org/sopt/pingle/data/datasourceimpl/local/PingleLocalDataSourceImpl.kt

@@ -55,6 +55,26 @@ class PingleLocalDataSourceImpl @Inject constructor(
        get() = pref.getString(GROUP_NAME, "") ?: ""
        set(value) = pref.edit { putString(GROUP_NAME, value) }

+    override var meetingCount: String


이거 단체 디테일 조회 API 있어서 그거 사용하면 될 듯 합니다 ~

app/src/main/java/org/sopt/pingle/domain/model/GroupListEntity.kt

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

jihyunniiii · 2024-02-25T06:21:55Z

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

+            android:layout_width="wrap_content"
+            android:layout_height="wrap_content"
+            android:layout_marginBottom="20dp"
+            android:gravity="center"


wrapcontent로 해줬으면 이거 안 줘도 되지 않나요?

엇 마자요 ㅎ 헤헤 지웠어염

jihyunniiii · 2024-02-25T06:22:50Z

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

+            android:id="@+id/tv_my_group_move_new_group"
+            android:layout_width="wrap_content"
+            android:layout_height="wrap_content"
+            android:layout_marginBottom="20dp"


상하로도 중앙 정렬이라 상하 제약 다 줬으면 빼도 될 것 같습니당

이거 빼면 리사이클러뷰 영역이 밑으로 주욱 늘어나서 얘가 안 보여염!!

jihyunniiii · 2024-02-25T06:24:22Z

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

+            android:id="@+id/layout_my_group_menu"
+            android:layout_width="wrap_content"
+            android:layout_height="0dp"
+            android:layout_marginEnd="7dp"


top margin 없서욤

여기 패딩 때문에 겹치는 영역인데 마진을 뭘로 해야 되는걸까여,,?!

jihyunniiii · 2024-02-25T06:26:00Z

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

+                <TextView
+                    android:id="@+id/tv_my_group_menu_share"
+                    android:layout_width="wrap_content"
+                    android:layout_height="wrap_content"
+                    android:layout_marginEnd="10dp"
+                    android:text="@string/my_group_menu_share"
+                    android:textAppearance="@style/TextAppearance.Pingle.Body.Med.14"
+                    android:textColor="@color/g_03"
+                    app:layout_constraintBottom_toBottomOf="parent"
+                    app:layout_constraintStart_toStartOf="parent" />


얘도 마진 end 빼고 좌 우 정렬을 초대코드 복사하기 텍스트에 맞추는 게 좋겠네용

저기 마진end는 의미가 없어서 지웠습니다! 좌우정렬은 어떤 식으로 맞춰야 하나요?? 메뉴가 공유랑 복사 칸 하나씩 묶여있어서요ㅜ

jihyunniiii · 2024-02-25T06:26:59Z

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

+        AllModalDialogFragment(
+            title = getString(R.string.my_group_modal_move_question, viewModel.getGroupName()),
+            detail = null,
+            buttonText = getString(R.string.my_group_modal_change),
+            textButtonText = getString(R.string.my_group_modal_back),
+            clickBtn = { chageToNewGroup() },
+            clickTextBtn = { }
+        ).show(supportFragmentManager, CHANGE_MODAL)


기존 모달 사용하면 마진이 다르지 않나용? 확인 하셨을까욤??

넹 확인했서요 AllModalFragment 같은 칭구를 하나 더 만들어서 띄우려구요! 테스트용으로 썼습니둥

jihyunniiii · 2024-02-25T06:30:58Z

좀 전에 단 코리 fda97c8 이 커밋까지만 보고 단겁니다 ㅠ..ㅠ

Doreminwoo

고생하셨어용~~!!!!!!!!

Doreminwoo · 2024-02-26T10:16:07Z

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

+            if (viewModel.getMyGroupIsOwner()) {
+                ivMyGroupNowOwner.visibility = View.VISIBLE
+            } else {
+                ivMyGroupNowOwner.visibility = View.INVISIBLE
+            }


코드가 한줄인경우 대괄호를 빼는게 예쁜거같아요!

Suggested change

if (viewModel.getMyGroupIsOwner()) {

ivMyGroupNowOwner.visibility = View.VISIBLE

} else {

ivMyGroupNowOwner.visibility = View.INVISIBLE

}

if (viewModel.getMyGroupIsOwner()) ivMyGroupNowOwner.visibility = View.VISIBLE

else ivMyGroupNowOwner.visibility = View.INVISIBLE

이게ㅜㅜ 저도 그렇게 쓰고싶은데 컨트롤 엘이었나 린트였나 돌리면 일케 바뀌더라구요.. if~else문에 braces 없으면 안 좋다 어쩌구 하면서•• 참나!!

저는 가독성 때문에 if, else 괄호 치는게 좋음 ㅋㅋ 개취인듯여 ~~

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

Doreminwoo · 2024-02-26T10:18:36Z

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

+    fun Logging() {
+        Log.e("localStorage", localStorgae.groupId.toString())
+        Log.e("localStorage", localStorgae.groupName)
+        Log.e("localStorage", localStorgae.meetingCount)
+        Log.e("localStorage", localStorgae.participantCount)
+        Log.e("localStorage", localStorgae.isOwner.toString())
+        Log.e("localStorage", localStorgae.groupKeyword)
+        Log.e("localStorage", localStorgae.code)
+    }


로그는 피알 올릴 때 삭제해주거나 팀버로 바꿔주는게 좋을 것 같아요!

헉스 ㅎㅅㅎ 바로 삭제하게쑴당

jihyunniiii

카페 도착해서 마저 달겠습니당

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

jihyunniiii

쏘굿도링 얘네만 수정해주세용

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

jihyunniiii · 2024-03-01T06:00:43Z

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

+                if (selectedMyGroup?.isOwner == true) {
+                    ivMyGroupSelectedOwner.visibility = View.VISIBLE
+                } else {
+                    ivMyGroupSelectedOwner.visibility = View.INVISIBLE
+                }


Suggested change

if (selectedMyGroup?.isOwner == true) {

ivMyGroupSelectedOwner.visibility = View.VISIBLE

} else {

ivMyGroupSelectedOwner.visibility = View.INVISIBLE

}

ivMyGroupSelectedOwner.visibility = if (selectedMyGroup?.isOwner == true) View.VISIBLE else View.INVISIBLE

이렇게 적을 수도 있답니다

오마이갓 쏘간단도리

jihyunniiii · 2024-03-01T06:03:22Z

app/src/main/java/org/sopt/pingle/util/component/MyGroupModalDialogFragment.kt

마이 그룹에서만 쓰는 거니까 마이 그룹으로 이동 ㄱㄱ 해주세염

jihyunniiii · 2024-03-01T06:05:29Z

app/src/main/java/org/sopt/pingle/util/context/ContextExt.kt

+    val intent = Intent(Intent.ACTION_SEND_MULTIPLE).apply {
+        type = SHARE_TYPE
+        putExtra(Intent.EXTRA_TEXT, shareContent)
+    }
+    startActivity(Intent.createChooser(intent, null))


Suggested change

val intent = Intent(Intent.ACTION_SEND_MULTIPLE).apply {

type = SHARE_TYPE

putExtra(Intent.EXTRA_TEXT, shareContent)

}

startActivity(Intent.createChooser(intent, null))

Intent(Intent.ACTION_SEND_MULTIPLE).apply {

type = SHARE_TYPE

putExtra(Intent.EXTRA_TEXT, shareContent)

startActivity(Intent.createChooser(this, null))

}

요렇게 써도 됩니둥

수정 완!!

jihyunniiii · 2024-03-01T06:05:57Z

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

뒤에 _20 넣어줘용 안 넣은 것들 다

jihyunniiii · 2024-03-01T06:07:04Z

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

이거 뭔가 다 12인데 디자인 측에서 잘못한 것 같기두,,,??? 한 번 여쭤봐영

이거 뭔가 다 12인데 디자인 측에서 잘못한 것 같기두,,,??? 한 번 여쭤봐영

이거 의도한 거 맞는 것 같기두 해여 그 왕관 visible했을 때 오른쪽 아래가 12면 아래 테두리가 살짝쿵 보이더라구욤??

아하 그래도 혹시 모르니 한 번 물어봐주삼요!!!

넹 피그마 코멘트 남겨놔쑴다~!

jihyunniiii

쏘굿도링

HAJIEUN02 added 10 commits February 22, 2024 15:54

[add] #194 String 리소스 추가

c06dc52

[add] #194 이미지 리소스 추가

fc7ae00

[add] #194 이미지 리소스 추가

5d57c48

[add] #194 String 리소스 추가

edb4951

[feat] #194 나의 단체 뷰 : UI 구현

7ddaad9

[feat] #194 나의 단체 뷰 : Dto, Entity 추가 및 pref에 저장

d364248

[feat] #194 나의 단체 뷰 : 나의 단체 리사이클러뷰 구현

6707090

[feat] #194 나의 단체 뷰 : 초대코드 복사 기능 구현

d3dce46

[feat] #194 나의 단체 뷰 : 현재 단체 단체장 여부 visibility 설정

8cb2205

[feat] #194 나의 단체 뷰 : 현재 단체 공유하기 구현

c293513

HAJIEUN02 added ✨ feat ✨ 🐹 지은 🐹 labels Feb 24, 2024

HAJIEUN02 added this to the 3차 스프린트 뷰 작업 milestone Feb 24, 2024

HAJIEUN02 self-assigned this Feb 24, 2024

HAJIEUN02 requested a review from a team as a code owner February 24, 2024 15:11

[chore] #194 ktlintFormat 적용

acb6eb3

Dan2dani reviewed Feb 25, 2024

View reviewed changes

[chore] #194 activity_my_group.xml 데이터바인딩으로 변경

fda97c8

[feat] #194 단체 변경 로직 구현

1efd17e

HAJIEUN02 added 2 commits February 25, 2024 15:08

Merge branch 'develop' into feat-my-group

c22fe8c

# Conflicts: # app/src/main/res/values/strings.xml

[chore] #194 conflict 해결 및 ktlintFormat 적용

46574d2

jihyunniiii reviewed Feb 25, 2024

View reviewed changes

Doreminwoo reviewed Feb 26, 2024

View reviewed changes

HAJIEUN02 added 4 commits February 26, 2024 21:28

Merge branch 'develop' into feat-my-group

2715e79

[chore] #194 나의 단체 뷰 : chip style 적용

74cfc62

[feat] #194 나의 단체 뷰 : 복사하기, 공유하기 확장함수 구현

7d16745

[feat] #194 나의 단체 뷰 : 나의 단체 뷰 모달 구현

51dec5d

HAJIEUN02 added 2 commits February 29, 2024 15:55

[chore] #194 나의 단체 뷰 : 코드리뷰 반영

c36b0d9

[chore] #194 나의 단체 뷰 : ktlintFormat 적용

45319c2

jihyunniiii requested changes Feb 29, 2024

View reviewed changes

[chore] #194 나의 단체 뷰 : 코드리뷰 반영

62b6a15

jihyunniiii reviewed Mar 1, 2024

View reviewed changes

HAJIEUN02 added 2 commits March 1, 2024 15:28

[chore] #194 나의 단체 뷰 : 코드리뷰 반영

9924c7c

[chore] #194 더보기 뷰 : 단체명 갱신

46cd733

jihyunniiii self-requested a review March 1, 2024 12:31

jihyunniiii approved these changes Mar 1, 2024

View reviewed changes

HAJIEUN02 merged commit 7969101 into develop Mar 1, 2024
1 check passed

jihyunniiii deleted the feat-my-group branch March 3, 2024 00:36

[feat] 나의 단체 뷰 구현 #201

[feat] 나의 단체 뷰 구현 #201

Conversation

HAJIEUN02 commented Feb 24, 2024 • edited Loading

Related issue 🛠

Work Description ✏️

Screenshot 📸

Uncompleted Tasks 😅

To Reviewers 📢

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

jihyunniiii commented Feb 25, 2024

HAJIEUN02 commented Feb 25, 2024 • edited Loading

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

Choose a reason for hiding this comment

jihyunniiii commented Feb 25, 2024

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

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

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

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

HAJIEUN02 commented Feb 24, 2024 •

edited

Loading

HAJIEUN02 commented Feb 25, 2024 •

edited

Loading