Skip to content

Upgrade Onfido sdk version to 6.15.2#6211

Merged
jasperhuangg merged 1 commit into
mainfrom
nat-onfido6.15.2
Nov 4, 2021
Merged

Upgrade Onfido sdk version to 6.15.2#6211
jasperhuangg merged 1 commit into
mainfrom
nat-onfido6.15.2

Conversation

@nkuoch

@nkuoch nkuoch commented Nov 4, 2021

Copy link
Copy Markdown
Contributor

@chiragsalian

Details

Upgrade Onfido sdk version to 6.15.2

Fixed Issues

$ https://github.com/Expensify/Expensify/issues/183732

Tests / QA Steps

On NewDot, try to add a verified business bank account. Make sure the Onfido flow (asking you to take some pics/videos from your phone) is working properly.

Tested On

  • Web
  • Mobile Web
  • Desktop
  • iOS
  • Android

Screenshots

Web

@nkuoch nkuoch requested a review from a team as a code owner November 4, 2021 17:07
@nkuoch nkuoch self-assigned this Nov 4, 2021
@MelvinBot MelvinBot requested review from jasperhuangg and removed request for a team November 4, 2021 17:07

@chiragsalian chiragsalian left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@jasperhuangg jasperhuangg merged commit 5ac1125 into main Nov 4, 2021
@jasperhuangg jasperhuangg deleted the nat-onfido6.15.2 branch November 4, 2021 23:51
@OSBotify

OSBotify commented Nov 4, 2021

Copy link
Copy Markdown
Contributor

✋ This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

@OSBotify

Copy link
Copy Markdown
Contributor

🚀 Deployed to staging by @jasperhuangg in version: 1.1.14-5 🚀

platform result
🤖 android 🤖 success ✅
🖥 desktop 🖥 success ✅
🍎 iOS 🍎 success ✅
🕸 web 🕸 success ✅

@OSBotify

Copy link
Copy Markdown
Contributor

🚀 Deployed to production by @AndrewGable in version: 1.1.15-15 🚀

platform result
🤖 android 🤖 success ✅
🖥 desktop 🖥 success ✅
🍎 iOS 🍎 success ✅
🕸 web 🕸 success ✅

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