-
Notifications
You must be signed in to change notification settings - Fork 5
[ADD] Share Extension Target 추가 #49
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
Conversation
Suyeon9911
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
수고하셨어용 ~~ 홍선배꽈리 ~~~
All File Checked 📝LGTM✨ No Errors or Warnings Found Good Job!
Generated by 🚫 Danger |
|
|
||
| import UIKit | ||
|
|
||
| @objc(ShareViewController) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
p3;
해당 코드가 의미하는 바는 무엇이고 왜 필요한건가요??
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@sujinnaljin
팀이 지향하는 바가 스토리보드 없이 UI를 구성하는 것이기에 스토리보드(MainInterface.storyboard)를 먼저 삭제했습니다.
기존에는 ShareViewController를 MainInterface.storyboard를 통해 팝업시켜주었습니다.
스토리보드가 사라졌기에 Share Extension에 접근시 가장 먼저 띄워줄 EntryViewController(NSExtensionPrincipalClass)를 지정해주어야합니다.
따라서 해당 컨트롤러를 NSExtensionPrincipalClass로 지정해주기 위해 @objc(ShareViewController)를 이용해 objc runtime에 접근 가능하게 해주었습니다!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
오호 그래서 저 위에 요렇게 설정되어있는거군요
<key>NSExtensionPrincipalClass</key>
<string>ShareViewController</string>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
네 맞습니다!!!
sujinnaljin
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
긋~
beansbin
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
홍선배 멋쪄요~!!
YoonAh-dev
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
멋지네요~👍
🟣 관련 이슈
🟣 구현/변경 사항 및 이유
저장 프로세스 기능구현을 위한 Share Extension Target을 추가했습니다.
commit : 19bff5d
🟣 PR Point
🟣 참고 사항