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

[#43] 캘린더 뷰 #47

Merged
merged 4 commits into from
Oct 4, 2021
Merged

Conversation

jeongchoyi
Copy link
Member

Description

  • 캘린더 뷰 컴포넌트 배치
  • dummy data 생성
  • AppMonth Enum 생성
  • KingFisher 설치
  • View Transition 연결을 위해 CoverUserPhotoVC에 Navi init 함수 추가
  • Calendar, Tracker Data Model struct 추가

image

Related Issue

close #43

### Description
- 캘린더 뷰 컴포넌트 배치
- dummy data 생성
- AppMonth Enum 생성
- KingFisher 설치
- View Transition 연결을 위해 CoverUserPhotoVC에 Navi init 함수 추가
- Calendar, Tracker Data Model struct 추가

### Related Issue
close team-popo#43
Copy link
Member

@hyun99999 hyun99999 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

data model 에 codingkey 만들지 않아도 괜찮을까여?

}

func setCell(tracker: Tracker) {

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

나는 cell 을 초기화 할때 사용한다고 생각해서 initCell 로 설정했는데 어떤 걸루 통일할까?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

아항... 그럼 다 init으로 통일하는걸로 합시돠 !!

static let cellSpacing: CGFloat = 12
static let cellWidth = ( screenWidth - collectionViewSpacing - cellSpacing * 4 ) / 5
static let cellHeight = cellWidth * 1.3
}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

기존의 코드도 이런식으로 한번 바꿔줘야겠어 👍 요거 리펙토링할게여

@hyun99999
Copy link
Member

hyun99999 commented Oct 2, 2021

어라 지금 봤는데 충돌났네염
충돌보니까 파일 이름이랑 xib 변수명 바꿔서 그런거긴 하던데 devleop 브랜치 pull 받고 충돌 해결하구 다시 푸시할 예정인가여?

@jeongchoyi
Copy link
Member Author

어라 지금 봤는데 충돌났네염 충돌보니까 파일 이름이랑 xib 변수명 바꿔서 그런거긴 하던데 devleop 브랜치 pull 받고 충돌 해결하구 다시 푸시할 예정인가여?

아뇨 그냥 깃헙에서 resolve할게요 !

@jeongchoyi jeongchoyi merged commit 59ac973 into team-popo:develop Oct 4, 2021
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.

[feat] 캘린더 뷰
2 participants