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

feat: 덕질 명함 생성 뷰 구현 (#449) #458

Merged
merged 16 commits into from
Apr 24, 2023

Conversation

hyun99999
Copy link
Member

@hyun99999 hyun99999 commented Apr 24, 2023

🌴 PR 요약

🌱 작업한 브랜치

🌱 작업한 내용

  • feat: 덕질 명함 생성 뷰 구현 #449

    • 덕질 명함 생성과 덕질 명함 앞면, 미리보기 적용 및 생성까지 구현하였습니다.
    • MainCardCell 에서 cardType 에 따라 명함의 종류를 바꿔서 보여주게 됩니다.
    • 명함을 지정할 때 cardType enum 형에 따라서 분기처리해서 보여주시면 되겠습니다.
  • fix: 이미지 업로드 속도 개선 #457

    • jpeg 으로 변경 후, compression quality 를 수정하여 업로드 속도를 개선하였습니다. 0.5로 조절 후에도 이미지 품질에 영향이 없고, 업로드 속도도 적합하다고 판단하여 적용하게 되었습니다.

📸 스크린샷

기능 스크린샷
덕질 명함

📮 관련 이슈

@hyun99999 hyun99999 added Hyungyu 🐯 현규 교수님 작업 Feat 새로운 기능 구현 labels Apr 24, 2023
@hyun99999 hyun99999 requested a review from dlwns33 April 24, 2023 07:59
@hyun99999 hyun99999 self-assigned this Apr 24, 2023
Copy link
Member

@dlwns33 dlwns33 left a comment

Choose a reason for hiding this comment

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

image

Comment on lines +95 to +116
@objc
private func tapSNSLabel() {
let appURL: URL?
let webURL: URL?

if cardData?.instagram != nil {
let instagramID = snsLabel.text ?? ""
appURL = URL(string: "instagram://user?username=\(instagramID)")
webURL = URL(string: "https://instagram.com/\(instagramID)")
} else {
let twitterID = snsLabel.text ?? ""
appURL = URL(string: "twitter://user?id=\(twitterID)")
webURL = URL(string: "https://twitter.com/\(twitterID)")
}
guard let appURL, let webURL else { return }

if UIApplication.shared.canOpenURL(appURL) {
UIApplication.shared.open(appURL, options: [:], completionHandler: nil)
} else {
UIApplication.shared.open(webURL, options: [:], completionHandler: nil)
}
}
Copy link
Member

Choose a reason for hiding this comment

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

아이고 선생님 이거 분기처리하기 빡셌을텐데 수고하셨습니다..

Copy link
Member Author

Choose a reason for hiding this comment

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

모든 순간이 분기였다..
image

@hyun99999 hyun99999 merged commit 65b6875 into TeamNADA:develop Apr 24, 2023
@hyun99999 hyun99999 deleted the feature/#449 branch April 24, 2023 08:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feat 새로운 기능 구현 Hyungyu 🐯 현규 교수님 작업
Projects
None yet
Development

Successfully merging this pull request may close these issues.

fix: 이미지 업로드 속도 개선 feat: 덕질 명함 생성 뷰 구현
2 participants