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

Share Extension 추가 및 앱에 사진 공유 기능 추가 #123

Merged
merged 6 commits into from
Nov 29, 2024

Conversation

dbqls200
Copy link
Collaborator

@dbqls200 dbqls200 commented Nov 28, 2024

📝 작업 내용

이번 PR에서 작업한 내용을 간략히 설명해주세요

  • 외부에서 앱에 사진 공유하기 위해, Share Extension 추가

  • 사진 공유할 때 로직은 아래와 같습니다.

    외부에서 앱으로 사진 공유
    -> UserDefault에 이미지 저장
    -> 저장되어 있는 공유된 이미지가 있으면, 앱 진입했을 때 경로 입력 창으로 넘어감.

  • ImageHandler 안에 있는 loadImage 함수 수정했습니다.

    • 공유해서 가져오는 이미지가 UIImage 타입인데, loadImage 함수가 PhotosPickerItem 타입을 받고 있어서 사용 불가능.

    AS-IS : 현재는 PhotosPickerItem 타입을 받는 loadImage 함수 / UIImage 타입을 받는 loadImage 함수가 각각 선언되어 사용되고 있음.

    • 확인해보니, 기존 loadImage 함수 내부에서도 PhotosPickerItemUIImage로 변환해서 사용 중이더라구요.

    TO-BE : 나중에 PhotosPickerItem를 UIImage로 변환하는 함수를 하나 만들고 지금 중복으로 작성되어 있는 loadImage 함수를 하나만 작성해둘 예정입니다!

스크린샷

💬 리뷰 요구사항

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

  • TMT 앱 타겟에 필요한 개발 / 릴리즈 프로비저닝 파일 다시 보내드릴게요.
    앱 그룹 설정 이전에 받았던 파일이라 오류가 있네용.

수정할 부분 공유할게요. 일단 지금까지 발견한 것들만 써볼게요 .. 더 있을 수도 있어유 🥲

[ 고쳐야할 점 ]

  1. 앱 실행 여부와 상관없이, 사진이 공유되면 업데이트 되어야 함. (이미 경로 시작 중인 경우 제외) (시나리오 1)
  • NotUploadedViewonAppear될 때만 해당 로직이 실행되니까 … 수정해야 함.
  1. 캡쳐 이후 바로 공유하기 하면 안 됨. (시나리오 2)
  2. 공유 완료되면, 우리 앱으로 자동 넘어가도록 하기.

[ 시나리오 1 ]
사진 올리고 입력 뷰까지 진입 → 갤러리 가서 사진 공유 → 앱 내 변화 없음. (당연함) → 앱 날렸다가 다시 들어오면, 공유했던 사진 뜸.

[ 시나리오 2 ]
사진 캡쳐 → 좌측 하단에 뜬 사진 눌러서 공유 → 앱 동작 없음 ㅜㅜ

- loadImage 함수가 매개변수 타입에 따라, (PhotosPickerItem / UIImage)로 중복되어 있음.
- PhotosPickerItem을 UIImage로 변환하는 함수 추가 예정
- loadImage 함수 하나로 두 타입 대응할 수 있도록 수정 예정
- 사진 공유 sheet에 앱 노출
- 외부에서 앱으로 사진 공유하면 UserDefault에 저장
- UserDefault에 저장된 사진 가져와서 경로 설정
@dbqls200 dbqls200 self-assigned this Nov 28, 2024
Copy link
Collaborator

@Minkyeong-Choi Minkyeong-Choi left a comment

Choose a reason for hiding this comment

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

Share Extension 어려운거였군요..! 수고하셨습니당!! 안되는 부분은 차근차근 고쳐봐요 저희

@dbqls200 dbqls200 merged commit c9facae into dev Nov 29, 2024
@dbqls200 dbqls200 deleted the feat/106-shared-extension branch November 29, 2024 07:18
@dbqls200 dbqls200 linked an issue Nov 30, 2024 that may be closed by this pull request
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

네이버 지도 캡처 화면 공유 기능 구현하기
2 participants