-
Notifications
You must be signed in to change notification settings - Fork 5
221116_스크럼
Dong-eun Shin edited this page Nov 17, 2022
·
1 revision
- 병학
- 3시까지 Firebase 활용 클래스 완성
- 탭바 컨트롤러 작업
- 애플 개발자 계정 인증서 공유
- 영욱
- 채팅방 목록 코디네이터 작업 완료
- 채팅방 목록 ViewModel 구현
- 준영
- 로그인 작업 (애플 개발자 계정이 지금 없어, 병학님께 인증서 공유 받으면 시작 예정)
- 앱 설정, 프로필 수정, 마이 프로필 코디네이터, 뷰모델 작업 예정
- 창묵
- 더미 데이터를 이용한 Cluster Annotation
- (버그가 발생 안하면) modal 창
- 동은
- 채팅 생성 화면, 프로필 상세 화면 VM 구현
- input output 패턴 적용해서
- 채팅 생성 화면, 프로필 상세 화면 VM 구현
- 의견: 공통적으로 사용하는 단어 영어로 사용할때 통일성이 필요하다.
- 결론: 아래와 같이 정의
채팅방 → chatRoom
그룹채팅방 → groupChatRoom
개인채팅방 → dmChatRoom
- 의견: 뷰 추가 및 오토레이아웃 관련 함수명 통일하는게 좋지 않을까
- 결론: 아래와 같이 정의
// Convention
private func addSubviews() { // 뷰 추가
view.addSubview(button)
view.addSubview(title)
configureConstraints()
}
private func configureConstraints() {
configureButton()
configureTtitle()
}
- 의견: 위에서 논의한 뷰 관련 함수를 Protocol로 관리하면 어떨까
- 결론:
- PR 머지할 때 project 파일에서 컨플릭트가 일어날 수 있어서 resolve 하는 과정에 신경을 써야한다.
- 병학
- PR comment에 대해 확인 표시가 필요할 것 같다. → 이모티콘으로 확인 표시
- PR 관련 시간 그라운드룰 수정 → 수정 후 슬랙에 말할 예정
- https://github.com/boostcampwm-2022/iOS06-NearTalk/pull/66
- 파이어베이스 활용 클래스 50% 구현
- 내일 오전에 파이어베이스 클래스 완성 후 PR 생성할 예정
- 영욱
- 코디네이터 패턴 적용
- 클린 코드 적용
- 준영
- 마이프로필 → 앱설정, 프로필 수정 코디네이터 작업
- 로그인은 인증서 문제로 잠시 중지. 병학님께 인증서 받으면 작업 예정
- 창묵
- Cluster Annotation 완료
- modal: UIPresentationController
- 다방 UI 참고
- 동은
- 오전에 공유한 Task 계속 진행중
-
private let VS private lazy var
- 동은 - 타이밍이 나눠지니까 성능에 도움이 되지 않을까 생각함
- 병학 - lazy var는 self를 쓰지 않으면 효과가 크지 않다고 생각함
- 창묵 - lazy var를 사용하면 성능이 좋아 질 수 있겠지만, 무조건 적으로 쓸 필요는 없다고 생각함
- 참고자료
- 애플 로그인 certification 파일 slack에 전달
- Flow Coordinator에 대해서 개별 공부 필요
Copyright © 2022 NearTalk