Closed dlrjswns closed 1 month ago
첫번째 코드리뷰였네요 수고하셨습니다! 🫡 완성된 기능은 아니라서 사소한 부분만 코멘트 달아봤어용
다만 PR이나 커밋을 작성하실 때, 더 자세히 작성이 필요할 것 같습니다. 전반적으로 커밋이
세팅 추가
UI 구현
이런식으로 모호하게만 작성되어 있고, 상세한 내용은 적혀있지 않아서 어떤 의도로 어떤 작업을 하셨는지 알기가 어려웠어요PR같은 경우에는, PR 템플릿과 컨벤션이 지정될 예정이니 만나서 설명드릴게요. 궁금하시다면 헤이딜러 개발팀 모두가 행복한 개발/PR관리 방법 7가지 이 링크를 한번 읽어보시면 도움될 것 같습니다.
요런거 진짜 너무 감사해요... 본격적으로 개발들어가기 이전에 함께 협업하는 부분이니만큼 블로그 읽어볼게요 ☺️☺️☺️☺️❤️
머지 완료되었으니 기존 feature/#10-socialLogin-dev
브랜치는 삭제합니다.
만약 애플 로그인 관련 추가 작업이 필요할 경우
feature/#10-socialLogin-master
브랜치에 PR 날려주시면 됩니다.주석 관련한 수정 사항이 몇개 있는데, 많진 않아서 직접 수정하고 머지할게용 아래 링크 읽어보세요 https://nshipster.com/swift-documentation/
나중에 코멘트 달린것들만 다시 확인해주세요
에구 감사합니다 👍👍 한번 읽어보고 이번 세션회의때 확실히 알고 넘어갈게요 !!! 부족한 저 때문에 고생많으십니다 😭😭
TODO: - @ChoiysApple 추후 애플 소셜로그인 구현 필요