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

Feature/friend view #124

Merged
merged 11 commits into from
Dec 9, 2022
Merged

Feature/friend view #124

merged 11 commits into from
Dec 9, 2022

Conversation

jinwoong16
Copy link
Collaborator

📕 Issue Number

Close #123

📙 작업 내역

구현 내용 및 작업 했던 내역

  • FriendView를 작성
  • FriendViewModel 작성
  • FriendQeustUseCase 작성
  • FriendCalendarUseCase 작성

📘 작업 유형

  • 신규 기능 추가
  • 버그 수정
  • 리펙토링
  • 문서 업데이트

📋 체크리스트

  • Merge 하는 브랜치가 올바른가?
  • 코딩컨벤션을 준수하는가?
  • PR과 관련없는 변경사항이 없는가?
  • 내 코드에 대한 자기 검토가 되었는가?
  • 변경사항이 효과적이거나 동작이 작동한다는 것을 보증하는 테스트를 추가하였는가?
  • 새로운 테스트와 기존의 테스트가 변경사항에 대해 만족하는가?

📝 PR 특이 사항

PR을 볼 때 주의깊게 봐야하거나 말하고 싶은 점

  • 여기도 세달치 정보를 가져오는 로직으로 시도했다가 너무 오래걸려서 한달치만 가져오는 것으로 변경했습니다
  • 그래도 오래걸럽니다.
  • VCR로 만나보시죠.
months.mp4
  • 세달치입니다.
a.month.mp4
  • 한달치입니다.



@jinwoong16 jinwoong16 added this to the 5주차 개발! milestone Dec 8, 2022
@jinwoong16 jinwoong16 merged commit 881f878 into develop Dec 9, 2022
@jinwoong16 jinwoong16 deleted the feature/FriendView branch December 9, 2022 09:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

FriendView
1 participant