실수로 #90 PR과 합쳐졌습니다.. 리뷰 이후 커밋에 대해 다시 리뷰 부탁드립니다

** 변경사항 **

• Dialog 추가
• Adpater를 data binding 방식에서 view binding으로 변경
• cid 목록을 효율적으로 쿼리하도록 SearchChatRemoteDataSource 수정

기능 변경사항

• 바텀 시트를 올리면 여러 채팅방을 볼 수 있음.
• 채팅방을 누르면 해탕 채팅방에 가입할 수 있음

UI 변경사항

• 채팅방을 표시하기 위해 ChatAdapter 추가
• 뷰홀더에는 item_chat_list_no_image.xml 사용
• Adapter는 뷰모델에서 생성하며, 이때 생성자에 클릭 리스너도 넣어줌,

Stram API 변경사항

• User 권한에 Join Chat을 추가함

앞으로 할 내용

• 채팅방에 접속하기 전 다이어로그 띄우기
• 작은 화면에서도 채팅방 정보 클릭했을 때 채팅방에 가입할 수 있도록 하기
• 채팅방 접속 로직을 data 모듈로 이전
• 마커 및 클러스터 마커에서 list 형태로 채팅방 id 받아오도록 고치기

작업 내용

• chatmodule 추가
• app 모듈에서 채팅방 목록 화면에서 새로운 모듈 사용
• data 모듈에서 채팅방, 유저 가져올 때 새로운 모듈 사용
• data 모듈에서 채팅방, 유저 추가할 때 새로운 모듈 사용
• 사용하지 않는 mapper, binding adapter, const 함수 및 변수 제거

UI 변경 사항

• 바텀시트의 채팅방 정보가 크게 보임

기능 변경 사항

• Configuration 변경하고 돌아와도 다이오그와 바텀시트가 유지됨

코드 변경 사항

• ViewModel에 pref와 chatClient 의존성 남아있던 것 발견하여 제거함
  • pref 호출 부분은 UserLocalRepository에서 담당함
  • chatClient 호출 부분은 SearchChatRemoteRepository에서 담당하도록 했음

화면

코드 수정 내용

• UserRepository의 companion object에 lastUserInfo를 추가해서 캐시로 사용함
• 유저 정보를 넘겨줄 때 우선 chatClient.getCurrentUser를 사용하고, 여기서 null이 반환된다면 캐시된 lastUserInfo를 반환하도록 함
• presentation layer에서는 UserInfo 모델을, data layer에서는 User 모델을 사용하도록 함

실수로 메인 브랜치에 만들어서 커밋 많음

코드 수정

• 프로그레스바가 로그인 성공시에만 사라지던 문제 해결
• API 26 이하? 에서 문제되면 theme 옵션 삭제
• network manager unregister할 때 dismiss해줌

Stream 수정사항

• User permission 추가

맡은 기능

• 김경연
  • 바텀시트 사용성 개선
  • LoginActivity 리팩터링
  • 로그아웃 기능 추가
• 박재홍
  • 지도 및 마커 스타일 커스텀
  • NetworkStateManager 구현
• 정철주
  • 백버튼 버그 수정
• 최우석
  • 채팅방 나가기 기능 추가 및 관련 버그 수정
  • 지도 configuration change 대응
  • 이미지 저장 기능 구현

공통

• 각 화면의 버그 수정

기능 구현

• 로그아웃이 성공한 경우 저장된 토큰을 지우고 로그인 화면으로 돌아간다.
• 로그아웃이 실패하면 아무것도 하지 않는다.
• 하는 김에 다이어로그 계속 뜨는 문제도 고쳤음

보완할 점

• 구글 로그인이 최근에 선택한 계정을 자동으로 선택해버려서 로그아웃한 소용이 없다

기능 변경사항

• 로그인 이력이 있는 사용자는 로그인 화면을 안보도록 조치함
• 서버에 정보가 없는 경우에만 구글 로그인 결과로 받은 값을 Stream에 저장하도록 함
  • 프로필 설정에서 유저 정보를 바꾼 이후 구글 정보로 덮어씌워지는걸 막음
• 구글 로그인 이후 Stream 로그인을 위해 잠시 대기해야하는데, 그 작업시간동안 프로그레스바를 띄워줌

구현 변경사항

• 로그인 로직을 data 모듈로 옮김

  • Stream에 로그인 요청을 보낼 때 콜백 형태의 API call만 작동하는 버그가 있었음
  • callbackFlow를 1차 시도했으나 실패해서 모두 콜백 형태로 바꿈
  • UI에서는 UserUiCallback을, Data단에서는 UserDataCallback을 사용하며 두 콜백간의 변환은 UserRepository에서 하고있음.

• repository로 주요 로직을 옮김

  • Stream에서 빈 User 객체를 준 경우 repository에서 실패로 판단
  • 첫 가입자인지 repository에서 판단하고 구글에서 받은 정보를 Stream으로 전송함

보완할 내용

• 콜백을 callbackFlow 형태로 바꾸기

실행화면

개선 전 / 개선 후

Close #72

채팅방을 삭제한 후 마지막 채팅방을 다시 지우려고 시도했더니 IndexOutOfBoundsException 발생.

2022-12-08 15:30:21.345 3940-3940/com.kiwi.kiwitalk E/AndroidRuntime: FATAL EXCEPTION: main
    Process: com.kiwi.kiwitalk, PID: 3940
    java.lang.IndexOutOfBoundsException: Index: 5, Size: 5
        at java.util.ArrayList.get(ArrayList.java:437)
        at java.util.Collections$UnmodifiableList.get(Collections.java:1356)
        at androidx.recyclerview.widget.ListAdapter.getItem(ListAdapter.java:150)
        at com.kiwi.kiwitalk.ui.home.ChatListViewAdapter.onBindViewHolder$lambda$1(ChatListViewAdapter.kt:35)
        at com.kiwi.kiwitalk.ui.home.ChatListViewAdapter.$r8$lambda$cbLPbk6ue49EmVcdidC8xmh96wQ(Unknown Source:0)
        at com.kiwi.kiwitalk.ui.home.ChatListViewAdapter$$ExternalSyntheticLambda1.onLongClick(Unknown Source:4)
        at android.view.View.performLongClickInternal(View.java:7952)
        at android.view.View.performLongClick(View.java:7902)
        at android.view.View.performLongClick(View.java:7920)
        at android.view.View$CheckForLongPress.run(View.java:29317)
        at android.os.Handler.handleCallback(Handler.java:883)
        at android.os.Handler.dispatchMessage(Handler.java:100)
        at android.os.Looper.loop(Looper.java:237)
        at android.app.ActivityThread.main(ActivityThread.java:8167)
        at java.lang.reflect.Method.invoke(Native Method)
        at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:496)
        at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1100)

• 더보기 너무 안보인다. 가로 2 x n 에서 1 x n으로 바꾸자

• 인원수 초과에 대응하자. ex) 채팅방을 아예 안보여주거나, 인원 초과 스낵바 표출

• configuration change때 다이어로그 터지는 것 대응

• 바텀시트에 프로그레스바 넣기

• 바텀시트가 최대로 펼쳐졌을 때 툴바를 가리지 않는 크기가 되도록

• 뒤로가기 누르면 바텀시트 접기

• 채팅방 들어갔다가 나오면 홈 화면으로 돌아가도록 하기 ← finishaffinity