Skip to content
This repository was archived by the owner on Jul 10, 2025. It is now read-only.

Conversation

mzorz
Copy link
Contributor

@mzorz mzorz commented May 28, 2020

Builds on top of #354

Related WPAndroid PR: wordpress-mobile/WordPress-Android#12031
(this means the WPAndroid PR points to this branch)

This PR adds a simple interface to signal listeners when the ComposeLoopFrameActivity has been abandoned without any Story being saved at all (i.e. pressing back on the camera preview screen while current Story in the StoryRepository has zero frames).

Comes in handy for listeners to do any cleanup they might need.

To test:

  1. open the demo app
  2. tap on the FAB + button
  3. tap BACK
  4. observe the Toast "Story has been discarded!" is shown (this means the listener interface worked)

@peril-automattic
Copy link

peril-automattic bot commented May 28, 2020

You can test the changes on this Pull Request by downloading the APK here.

Base automatically changed from feature/wp-stories-part81-add-file-result to feature/273-save-frame-service-base June 19, 2020 20:23
Copy link
Contributor

@jd-alexander jd-alexander left a comment

Choose a reason for hiding this comment

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

Thanks for these changes @mzorz Tested and working! LGTM 🚢

@jd-alexander jd-alexander merged commit c4f004d into feature/273-save-frame-service-base Jun 19, 2020
@jd-alexander jd-alexander deleted the feature/wp-stories-part82-on-discard-listener branch June 19, 2020 23:04
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants