Skip to content

Conversation

eduardoyoti
Copy link
Collaborator

@eduardoyoti eduardoyoti commented Jul 24, 2018

With this change we are hoping to avoid the user clicking the browser on the app chooser dialog, which could be the reason of the behaviour seen for this ticket.

https://lampkicking.atlassian.net/browse/ED-545

ED-545

Verification Tests

  • Yoti app not installed -> website

  • Make sure Yoti is not installed

  • Install the SampleApp2

  • Click on the "Share my phone" button

  • Check that the website is showed

  • Yoti app installed -> Yoti

  • Install Yoti app

  • Open the SampleApp2

  • Click on the "Share my phone" button

  • Check that the Yoti app is opened and show the share confirm screen

@lampkicking/android-dev

Copy link
Contributor

@cquemin cquemin left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@RobertLK
Copy link
Collaborator

1st scenario fails (shows "something went wrong") because we already checked this in startScenario and fired the error callback. Need to decide what behaviour we expect here.

# Conflicts:
#	yoti-sdk/src/main/java/com/yoti/mobile/android/sdk/kernelSDK/KernelSDKIntentService.java
Improving "Yoti app not installed" case in Sample-app.

ED-545
@RobertLK
Copy link
Collaborator

Tests passing now (I can't leave reviews or check the checkboxes so I might be missing some rights 😞 )

@eduardoyoti eduardoyoti merged commit ab5f830 into master Sep 19, 2018
@eduardoyoti eduardoyoti deleted the ED-545 branch September 19, 2018 16:10
mariasenosiain added a commit that referenced this pull request Apr 6, 2022
# This is the 1st commit message:

Update publication scripts & gradle configs.

Related to YM-26133

# The commit message #2 will be skipped:

# Set up gradle properties to work with aar.gradle 2.2.2
#
# Related to YD-26133

# The commit message #3 will be skipped:

# Add configurable app package
#
# YM-25986

# The commit message #4 will be skipped:

# Bump version and update CHANGELOG
#
# YM-25986
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.

4 participants