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.

Sign up for GitHub

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

Jump to bottom

feature/byeongyun 6 #190

Open

ITlearning wants to merge 6 commits into main from feature/byeongyun-6

Collaborator

ITlearning commented Oct 14, 2021

아직 문제가 많은 앱입니다..
먼저 위쪽 잔디는 한달을 기준으로 컬렉션 뷰 셀을 생성할 수 있게 설정해두었습니다.
앱 처음 시작 시 딜레이를 3초를 주어 서버 통신을 하고 띄워주는 방식을 택하였습니다.
그리고 API가 불안정해서 곳곳 커밋이 업데이트 되지 않는 점이 있었습니다..
그래프도 일별 이외에는 서버에서 가져오는 방식을 취합니다.
현재 API 트래픽 초과로 불안정한건지,,업데이트가 안될 때가 있습니다.
그래프 아래에 초록색이 남는것도 추후 보완하겠습니다..ㅠㅅ ㅠ
이번주에 더 보완하겠습니다. 죄송합니다 ㅠ
(이제 자러갑니다 총총,,)(안녕,,)

ITlearning added 6 commits

October 14, 2021 00:57


          #183 - 프로젝트 기본 세팅

7303f67


          #186 - 디테일 구성

5340edf


          #187 - 그래프 선택하기 버튼 활성화

fe726c5


          #188 - Presenter 추가 수정

d15076e


          함수 이름 변경

afdc45e


          #189 - 코드정리

1d36b48

ITlearning requested review from SH4CK3RS and della-padula

October 14, 2021 23:02

SH4CK3RS reviewed

View reviewed changes

Byeongyun/Practice6/Practice6/View/GraphView.swift

+                      let layer = CAShapeLayer()
+                      let path = UIBezierPath()
+                      let xOffset: CGFloat = self.frame.width / CGFloat(values.count)

Collaborator

SH4CK3RS Oct 18, 2021

draw에서는 frame보다 rect값을 활용해보세요

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