Skip to content

Commit

Permalink
[native] Combine invokeAsync calls at the end of initializeCryptoAccount
Browse files Browse the repository at this point in the history
Summary:
I think these were accidentally split up in D11232. In the first revision of that diff, it made sense to separate them. By the final revision, it no longer made sense, but the split was left there.

Depends on D13526

Test Plan: Confirm that the app builds and that `initializeCryptoAccount` still executes successfully

Reviewers: varun, will

Reviewed By: will

Subscribers: tomek

Differential Revision: https://phab.comm.dev/D13527
  • Loading branch information
Ashoat committed Sep 30, 2024
1 parent 8c10b49 commit 8f8315d
Showing 1 changed file with 1 addition and 3 deletions.
4 changes: 1 addition & 3 deletions native/cpp/CommonCpp/NativeModules/CommCoreModule.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -685,10 +685,8 @@ jsi::Value CommCoreModule::initializeCryptoAccount(jsi::Runtime &rt) {
promise->reject(error);
return;
}
promise->resolve(jsi::Value::undefined());
});

this->jsInvoker_->invokeAsync(
[=]() { promise->resolve(jsi::Value::undefined()); });
};
try {
this->cryptoThread->scheduleTask(cryptoJob);
Expand Down

0 comments on commit 8f8315d

Please sign in to comment.