Skip to content
This repository has been archived by the owner on Feb 4, 2022. It is now read-only.

Sharing on iOS #233

Open
wants to merge 18 commits into
base: develop
Choose a base branch
from
Open

Sharing on iOS #233

wants to merge 18 commits into from

Conversation

999eagle
Copy link
Member

@999eagle 999eagle commented Apr 5, 2019

This PR implements sharing images on iOS directly to Openbook.

@999eagle 999eagle changed the title WIP: Sharing on iOS Sharing on iOS Apr 6, 2019
@999eagle
Copy link
Member Author

999eagle commented Apr 6, 2019

In the current state, sharing on iOS works fine from other apps directly to Openbook. There probably are some edge cases when it doesn't work and proper error handling is missing in some places as well.

@lifenautjoe
Copy link
Member

Hi Sophie! Tried sharing from the gallery to Openbook and nothing happened, perhaps this is one of those edge cases? It allows to add text as well over there 🤔

@999eagle
Copy link
Member Author

999eagle commented Apr 6, 2019 via email

@lifenautjoe
Copy link
Member

lifenautjoe commented Apr 6, 2019 via email

@999eagle
Copy link
Member Author

999eagle commented Apr 6, 2019 via email

@uiboy
Copy link
Member

uiboy commented Apr 24, 2019

@lifenautjoe is this ready to go? was just looking at all the Open PRs

@999eagle
Copy link
Member Author

@uiboy there's still a problem with this as it works perfectly fine in the iOS simulator, but when @lifenautjoe tried it on a physical device, nothing happened.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants