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

[루시] 5주차 미션 #22

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

[루시] 5주차 미션 #22

wants to merge 1 commit into from

Conversation

kimsoomin20221789
Copy link

📌 관련 이슈

#21

✨ 이슈 내용

Moya를 사용한 API 네트워크 통신

📸 스크린샷(선택)

📚 레퍼런스 (또는 새로 알게 된 내용) 혹은 궁금한 사항들

Copy link
Member

@iiuoon iiuoon left a comment

Choose a reason for hiding this comment

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

수고하셨습니다 코드 리뷰 확인해주세요~!

Comment on lines +47 to +50
apiButton.snp.makeConstraints { make in
// make.top.equalTo(testTextview.snp.bottom).offset(10) // 텍스트 뷰 아래로 10포인트 떨어짐
make.centerX.equalToSuperview()
}
Copy link
Member

Choose a reason for hiding this comment

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

apiButton만 두번 되어 있어요

Comment on lines +22 to +28
var testTextview: UITextView {
let textView = UITextView()
textView.frame = CGRect(x: 20, y: 100, width: 300, height: 200)
textView.font = UIFont.systemFont(ofSize: 16)
textView.text = "안녕하세요, UITextView 예제입니다."
return textView
}
Copy link
Member

Choose a reason for hiding this comment

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

프로퍼티 선언이 잘못 돼서 시뮬레이션이 튕겨요!
값을 불러오는 거니까 UILabel을 사용하는 게 더 좋을 것 같아요

@iiuoon iiuoon added the 🌟 루시 루시's work label Nov 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🌟 루시 루시's work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants