[8주차/레오] 키워드 과제 제출합니다.

[yongyong213]

✅ 워크북 체크리스트

• 모든 핵심 키워드 정리를 마쳤나요?
• 핵심 키워드에 대해 완벽히 이해하셨나요?
• 이론 학습 이후 직접 실습을 해보는 시간을 가졌나요?
• 미션을 수행하셨나요?
• 미션을 기록하셨나요?

✅ 컨벤션 체크리스트

• 디렉토리 구조 컨벤션을 잘 지켰나요?
• pr 제목을 컨벤션에 맞게 작성하였나요?
• pr에 해당되는 이슈를 연결하였나요?(중요)
• 적절한 라벨을 설정하였나요?
• 파트장에게 code review를 요청하기 위해 reviewer를 등록하였나요?
• 닉네임/main 브랜치의 최신 상태를 반영하고 있는지 확인했나요?(매우 중요!)

📌 주안점

[jiyoung02]

LGTM! 수고하셨습니다 😊
로그인 기능과 전반적인 화면 흐름이 정말 잘 구현되어 있습니다!
아래는 함께 개선해보면 완성도를 더 높일 수 있는 부분들이니 참고해주세요.

1. 미니 플레이어 클릭 리스너 세분화
   현재 MainActivity의 하단 미니 플레이어는 전체 영역에만 클릭 이벤트가 부여되어 있어, 어느 위치를 눌러도 SongActivity로 이동하고 있습니다.각 버튼(재생, 일시정지, 다음 곡 등)에 개별 클릭 리스너를 부여하여 메인 화면에서도 음악 컨트롤이 가능하도록 개선해주세요!

2. HomeFragment 의 Runnable 객체 선언 위치를 확인해주세요!
   현재 Runnable이 Fragment 초기화 시점에서 선언되어 있어, 코드 구조상 빌드가 불가능해 보입니다 ㅠㅠ 레오의 컴퓨터에서는 빌드가 되는 상황인지 궁금합니다!
   만약 빌드를 위해 별도의 처리를 하셨다면, 그 변경사항도 깃허브에 함께 올려주세요!

[jiyoung02]

LGTM! 수고하셨습니다 😊
API 연결 처음 시도하실 때 어려움이 많았을 텐데, 정말 잘 해주셨어요!!
(남은 LoginApi연결도 잘 마무리해주실 거라 믿습니다 🙌)

다만 SignUpActivity에서 여전히 Firebase를 직접 호출하는 코드가 남아 있어
DB가 이중으로 호출되는 구조입니다.
Firebase와 서버를 동시에 호출하면 불필요한 성능 저하가 발생할 수 있으니,
특별한 이유가 없는 한 동시 호출은 피해주세요!