FE-957: Requests stuck verifying
Summary of changes made
-
There was an unhandled case where nodes not being ready would prevent the user lookup from even running, but the callback was not notified. The callback is now correctly notified of an error.
-
There was an unhandled case where requests still verifying would be marked as failed when the network follower stopped successfuly, but this would not happen if stopNetworkFollower throws an exception. Requests are now marked as failed even when stopNetworkFollower() throws an exception.
-
Added a new request status, "Received" instead of incoming requests being marked as "Verifying". Requests are now only updated to "Verifying" when verification starts.
-
On app launch, before the requests listener would be registered, requests that have "Verifying" status are now updated to "Verification failed" so they can be manually retried.
Merge request reports
Activity
requested review from @ahmedshehata01
assigned to @kam.elixxir
added 94 commits
-
54323da4...83f5f2a3 - 93 commits from branch
development
- d57a0b67 - Merge branch 'development' into 'FE-957_stuck_verifying'
-
54323da4...83f5f2a3 - 93 commits from branch
mentioned in commit 73deb3f6