Discussion

디코(완료), 카톡, 피그마 협의사항 옮기기

5/30 20시 이종성, 김태영

5/30 21시 이종성

Fake 데이터는 Commit에 포함하지 않도록 하고, 개발 시에만 사용합니다 (Fake 관련 import 구문도 포함되지 않도록 해주세요.)
Commit 시에는 Fake가 아닌 사용하는 Repository 인터페이스 이름 뒤에 Impl이 붙은 빈 껍데기 구현체로 변경해서 올려주시면 됩니다.
이렇게 할 경우, Repository가 구현되었을 때, UI 레이어에서 별도로 추가 작업을 하지 않아도 됩니다.
또한, 다른 사람이 코드 리뷰시 코드를 실행해 볼 수 있도록 PR 본문에 사용하신 Fake 구현체 코드를 함께 첨부해주세요.

5/31 12시 최윤찬

Ui를 짜다 보니 categoryUiSate data class의 추가 되지 않는 파라미터 값이랑 변경 해야할 파라미터 값이 있어서 변경하겠습니다.
해당 data class를 쓰는 사람이 저밖에 없어서 저 pr 올릴때 적용하면 될것 같습니다. 그래서 제 브랜치에서 변경해서 한번에 올릴게용~~
[변경사항]

val name: Int -> val name: String
val icon: String 추가

6/1 15시 이종성

6/3 12시 김태영

30라인의 toObject 메서드를 사용하려면 class의 인자가 없는 기본 생성자를 필요로 하여 user, meeting 등 firestore에서 가져올 때 쓰는 모든 data class에 기본 생성자를 추가하도록 협의하였습니다

6/3 15시 김태영
fitler - Cost의 name값이 String이 아닌 Int 타입이여서 수정했습니다. "3만원 이하", "3~5만원" 등의 텍스트가 들어갈 예정입니다.

6/3 17시 김태영, 최윤찬, 이종성

모임 검색 시 모임 리스트 item 내부에 호스트의 온도가 있는데, Meeting data class 에서 누락되어 추가합니다 (최윤찬, 김태영)
이에 따라, 미팅 생성 시 자동으로 호스트의 온도가 DB에 저장되도록 코드 수정중입니다.

AppSettingRepository의 getPrivacyInfo() : String로 개인정보처리방침 Notion url을 가져올 수 있도록 DB에 추가할 계획입니다 (이종성, 김태영)

6/3 18시 이종성

알림 설정의 경우, dataStore에 접근하지 않고, Presentation Layer 단에서 Context를 이용해 각 채널의 On/Off 정보를 바로 알 수 있으므로 AppSettingRepository에 저장하지는 않겠습니다.
현재 AppSettingRepository에는 가입 시 체크하는 아이디 저장 여부와 사용자가 로그인 했을 때 저장하는 UserDocumentID 2가지만 관리합니다.

6/3 20시 김태영

cost 데이터를 firestore로 부터 전달 받을 때, 아래와 같이 잘 정렬되어 올 때도 있지만, type과 name의 순서가 뒤바껴 올 때가 있어 에러가 발생합니다.
따라서, name과 type을 모두 string으로 하는 부 생성자가 필요하여 추가하였습니다.
- 정상 - [Cost(name=3만원 이하, type=1), Cost(name=3-5만원, type=2), Cost(name=5-10만원, type=3), Cost(name=10만원 이상, type=4)]
- 가끔 - [Cost(type=1, name=3만원 이하), Cost(name=3-5만원, type=2), Cost(name=5-10만원, type=3), Cost(name=10만원 이상, type=4)]

6/5 18시 김태영

[팀 규칙 추가]
- 모든 pr은 생성 시점으로부터 48시간 안에 리뷰와 수정작업을 거친다.
  -> 48시간이 지난 시점에서 반영되지 않은 필수 피드백 사항은 추후에 본인이 수정작업을 해야합니다.
  (그러니 꼭 제한된 시간 안에 수행해주세요)

6/5 20시 이종성

6/6 23시 김태영

준식님이 별도의 브랜치에서 User Data Class에 uuid 값을 추가한 뒤 PR을 올리도록 협의되었습니다.
지금까지 dev에 merge된 코드에서 User를 사용하는 코드는 준식님이 수정작업을 해주시지만, 현재 작업 중인 브랜치(dev에 merge되지 않은 브랜치)에서 User를 사용하는 코드는 이후 pull 받을 때, 각자 충돌 해결해주시면 되겠습니다!

6/7 17시 이종성

각 Fragment에서 데이터를 사용하는 것은 XxxUiState -> 각 화면에서만 사용하는 data class / 전달 과정에서 사용하는 data class 네이밍은 XxxUi -> 공용

6/11 16시 김태영, 이종성

notification 권한 알림은 MainActivity의 splash 화면 띄울 때 같이 요청한다.
다만, 사용자가 수락하든 거부하든 notification 관련 로직의 런타임 이슈가 없어야한다. (if문을 통해 권한이 있는지 체크하는 로직 필수)
앱을 처음 깐 경우, MainActivity에서 알림 권한이 왜 필요한지 dialog로 설명해주기 이후 요청받기 -> 거부해도 상관 없음

6/13 10시 김태영

Label Icon에서 bitmap은 로딩 속도(라벨 생성 속도), 이미지 크기 등의 이슈가 있어 일반 Marker Icon(drawable)로 변경되었습니다.

6/26 11시 김태영

저희가 모두 공통 텍스트 스타일을 사용했기 때문에 text_styles.xml 파일에서 textColor를 지정하여 다크모드에 대한 텍스트 색상은 한 번에 대응하려고 합니다.

Provide feedback