Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

9/6 daily scrum #64

Open
hhhminme opened this issue Sep 6, 2021 · 2 comments
Open

9/6 daily scrum #64

hhhminme opened this issue Sep 6, 2021 · 2 comments
Assignees

Comments

@hhhminme
Copy link
Collaborator

hhhminme commented Sep 6, 2021

  1. 지난 데일리 스크럼부터 지금까지 내가 완수한 것이 무엇인가
  2. 다음 데일리 스크럼까지 내가 하기로 한 것이 무엇인가
  3. 현재 장애가 되고 있는 것(곤란하고 어려운 것)이 무엇인가
@hhhminme
Copy link
Collaborator Author

hhhminme commented Sep 6, 2021

  1. 레벨 업 기능 코드 확인하고 졸업 기능을 엮었습니다. webhook events 활용을 너무나 잘해주어서 놀랐습니다. 하루에 2번 커밋 되는 요구 사항에 존재하는 timezone issue를 잘 해결해 있더군요. UI쪽 생명주기도 잘 다뤄주고 분기 처리와 모듈화가 잘되어 있어 가독성이 좋아 작업이 빠르게 끝났습니다. 이후 기능들은 제가 이번 주 동안 테스트해보며 졸업 기능이 잘 되는지 파악해보겠습니다. 고생 너무나 많으셨습니다.

또한 병아리 이름이 herming으로 지정되어 있는데 새로운 병아리를 받을 때마다 이름을 넣어주면 졸업시킬 때마다 우후죽순으로 병아리가 늘어날 것 같은데 이 부분은 회의 때같이 고민하는 게 좋을 것 같습니다. 그리고 졸업 시 아니요를 누르면 home_fragment를 죽여놔서 뒤로 가기 하면 앱이 꺼지더군요. 이 부분도 확인하고 수정하겠습니다.

마지막으로 #63 이슈가 있습니다. 해당 레포는 Public이고 main branch에 push 했습니다. 로그아웃 후 다른 레포에서도 push를 해보며 확인해보겠으나 기현 님 @kim1387 도 한번 확인해 주셨으면 좋겠습니다.

  1. 개강하니 날씨도 좋고 교수님들도 뵙고 동기들도 봐서 기분 좋네요😁

@kjsong99
Copy link
Collaborator

kjsong99 commented Sep 6, 2021

1.api 데이터 불러와서 레벨업 하는 기능 적용과 코드 가독성을 높이기 위해 모듈화를 했습니다
2.에러 해결 및 병아리 있는 지 확인 후 병아리 선택창 뜨게 하는 코드 작성
3.처음에 로그인 후 프로필 창을 불러오는 과정에서 username이 기본 값으로 들어가 있어 약간 신경이 쓰여서 수정할 예정입니다
로그인 처음 하는 경우 새로 고침을 누르지 않으면 뷰에 반영이 안됩니다
count가 0에서 1로 증가하는 경우 progress가 두번 진행됩니다 그 이후에는 정상적으로 증가하고 0에서 한번에 1이 아닌 다른 수로 증가되면 에러가 발생하지 않습니다

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants