[$500] Upgrade OnFido SDK to 8.0.0

[michaelhaxhiu]

If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!

---

Expected Result:

• OnFido needs to be updated to the newest version of the SDK (6.20.0), and so we need to upgrade accordingly -- full details of the new release are here https://github.com/onfido/onfido-sdk-ui/releases/tag/6.20.0.
• We need to do this because it's a prerequisite to troubleshooting/resolving potential issues that arise in their SDK
• Here's an example of where we upgraded the SDK in the past Upgrade Onfido sdk version to 6.15.2 #6211

Please update the Onfido UI SDK and Onfido React Native SDK in E/App

### Fixed in 6.20.1

- Public: Fix useLiveDocumentCapture onComplete not working (cross device capture `variant` missing)

---

Actual Result:

OnFido SDK isn't updated to the latest.

Workaround:

Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Yes.

Platform:

Where is this issue occurring?

• Web

Version Number:
Expensify/Expensify Issue URL: https://github.com/Expensify/Expensify/issues/206694

View all open jobs on GitHub

[melvin-bot]

Triggered auto assignment to @jboniface (AutoAssignerTriage), see https://stackoverflow.com/c/expensify/questions/4749 for more details.

[melvin-bot]

Triggered auto assignment to @marcaaron (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

[michaelhaxhiu]

@marcaaron can you help confirm that this is eligible for External? @amyevans mentioned that it's likely good for external but just wanted to double check as a part of the standard flow of creating a new GH in E/app (and bc the historic example of updating the SDK was done by internal engineers it seems?)

[marcaaron]

OnFido needs to be updated to the newest version of the SDK (6.20.0)

Where does it need to be upgraded? Everywhere or only New Expensify?

[marcaaron]

Agree this can be External as it's just updating a library. But the flow needs to be tested still so maybe we can make sure QA is done correctly and there are no regressions after updating.

[melvin-bot]

Triggered auto assignment to @MitchExpensify (External), see https://stackoverflow.com/c/expensify/questions/8582 for more details.

[MitchExpensify]

Upwork Job

[melvin-bot]

Triggered auto assignment to Contributor-plus team member for initial proposal review - @rushatgabhane (Exported)

[melvin-bot]

Triggered auto assignment to @marcochavezf (Exported), see https://stackoverflow.com/c/expensify/questions/7972 for more details.

[amyevans]

Where does it need to be upgraded? Everywhere or only New Expensify?

I just searched for where we're using the package, and it looks like it's in Web-Secure in addition to App. So perhaps this should stay internal after all?

[michaelhaxhiu]

cc @nkuoch or @chiragsalian for your input too, since you both were involved in the last SDK update for OnFido. Seems we still are uncertain about whether this is internal or external.

[marcochavezf]

Not overdue, waiting for proposals

[michaelhaxhiu]

Should we be doubling price of this job @MitchExpensify ?

[MitchExpensify]

Yes I will today @michaelhaxhiu

[MitchExpensify]

Doubled! Upwork Job

[Regaron]

Proposal

Looking at the change logs there doesn’t seem to be any breaking changes. So we can just update the packages and test for regressions.

[jayeshmangwani]

Proposal

it is a straightforward change of the updating package, need to update to the latest version of onfido-sdk-ui, as at this time it is 8.0.0, and then need to test in web and desktop , that update the package is working fine or not

[melvin-bot]

Current assignee @MitchExpensify is eligible for the External assigner, not assigning anyone new.

[melvin-bot]

Triggered auto assignment to Contributor-plus team member for initial proposal review - @Santhosh-Sellavel (Exported)

[melvin-bot]

Current assignee @marcochavezf is eligible for the Exported assigner, not assigning anyone new.

[michaelhaxhiu]

There is no C+ on this issue since Rushat un-assigned himself, so I'm triggering re-assignment.

[twisterdotcom]

@jayeshmangwani is right, this is now 8.0.0. Updated the issue title and our internal tracker.

[michaelhaxhiu]

This SDK seems to be updating pretty frequently, and consecutively each week... is this the norm? If so, I wonder if this is best handled internally after all?

[nkuoch]

I agree. I think whoever does Web-Secure internally could do App at the same time - it's not much more work.

[Santhosh-Sellavel]

Is this still external? or internal? @michaelhaxhiu

[michaelhaxhiu]

@Santhosh-Sellavel I'm working on making this internal right now, based on the comments above.

[michaelhaxhiu]

Sorry for the confusion on this GH @jayeshmangwani and @Regaron, we are closing this external job and will fix this internally instead. Thanks for your interest, feel free to browse other available jobs here