Automattic / newspack-ios

Newspack for iOS
https://newspack.pub
GNU General Public License v2.0
8 stars 4 forks source link

Share Extension Part Three #102

Closed aerych closed 4 years ago

aerych commented 4 years ago

Refs #91

This PR is part three of a four part series adding Share Extension support for sharing images to Newspack. In this PR we do the following:

This is another PR where a large portion of the changes are boiler plate, or due to incorporating existing production code from another app. New changes that need a keen eye are in:

The following files are slightly modified from their WPiOS counterparts, mostly to remove code that is not relevant to Newspack at this time. A passing glance wouldn't hurt but willing to trust the previous implementation.

To test:

Scenario 1: Logged out

Prep: Be logged out of Newspack. Launch the share extension with the Photos app.

Scenario 2: Logged in

Prep: Be logged in to Newspack with several stories. Launch the share extension with the Photos app.

Scenario 3: Confirm cast shadows

Scenario 4:

Example Screens:

Simulator Screen Shot - iPhone SE (2nd generation) - 2020-09-04 at 10 15 46 Simulator Screen Shot - iPhone SE (2nd generation) - 2020-09-04 at 10 15 53 Simulator Screen Shot - iPhone SE (2nd generation) - 2020-09-04 at 10 15 58 Simulator Screen Shot - iPad Pro (9 7-inch) - 2020-09-04 at 10 14 09

aerych commented 4 years ago

a potential good enhancement would be to make the "Story Picker" ViewController highlight the currently selected Story

Excellent idea.

peril-automattic[bot] commented 4 years ago

You can trigger an installable build for these changes by visiting CircleCI here.

aerych commented 4 years ago

Thanks @jleandroperez ! Changes made. One last quick peek for paranoia sake?

Screen shot showing the selected cell: Simulator Screen Shot - iPhone SE (2nd generation) - 2020-09-04 at 13 03 36 It's a little different from the main app, but the main app is probably going to change too so this is a good placeholder.

aerych commented 4 years ago

Thank you @jleandroperez !!