Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Verification does not save progress #218

Open
Claire-Bear opened this issue May 9, 2022 · 2 comments
Open

Verification does not save progress #218

Claire-Bear opened this issue May 9, 2022 · 2 comments
Assignees
Labels
bug Something isn't working M: asset-services Module: Asset services (Rust) M: web-client Module: Web client app (Angular)

Comments

@Claire-Bear
Copy link
Contributor

Bug description:
After verifying my profile where the screen shows "Status: In progress" if I press the back button before I press the refresh button, then I have to restart the whole onfido process from the start.
if I press refresh before I press the back button then my profile will be saved.
Even when I close and reopen the app it still prompts so this is not a refresh issue.

Rule:
If a user completes the KYC and reaches the point where they are awaiting their status of the submission to Ofido. No matter where they navigate to on the app if they click to return to the verification is should allow them to see the final ofido verification status until they save.
image

@Claire-Bear Claire-Bear added the bug Something isn't working label May 9, 2022
@PiDelport
Copy link
Collaborator

PiDelport commented May 10, 2022

This is not necessarily a small change.

We could adjust things so that in-progress Onfido checks have their state persisted as part of the overall wallet session state, but that complicates the handling of this significantly, and it's not really what we want to in the longer term: this state in general should really be managed by a KYC service module as part of the Asset Services.

@PiDelport PiDelport added M: web-client Module: Web client app (Angular) M: asset-services Module: Asset services (Rust) labels May 10, 2022
@Claire-Bear
Copy link
Contributor Author

@Brianspha Can we focus on some of the other items rather as Pi said this one might be a bigger issue and it was further down the list in terms of priority

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working M: asset-services Module: Asset services (Rust) M: web-client Module: Web client app (Angular)
Projects
None yet
Development

No branches or pull requests

3 participants