Skip to content
This repository has been archived by the owner on Mar 17, 2023. It is now read-only.

Return canceled on back #74

Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open

Return canceled on back #74

wants to merge 9 commits into from

Conversation

thirstycoda
Copy link

There are three scenarios that are useful to know:

  1. The user clicked done after making modifications to the photo
  2. The user clicked done but did not make any changes
  3. The user clicked back

Currently it is not possible to differentiate between 2 and 3. This pull request returns RESULT_CANCELED when the back button is pressed so differentiation is possible

@iamutkarshtiwari
Copy link
Owner

@thirstycoda Thanks for your great work! Could you please resolve these conflicts?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@binSaed binSaed binSaed approved these changes

@iamutkarshtiwari iamutkarshtiwari Awaiting requested review from iamutkarshtiwari

At least 1 approving review is required to merge this pull request.

Assignees

@thirstycoda thirstycoda

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@thirstycoda @iamutkarshtiwari @binSaed