Page MenuHomePhabricator
Differential D11470

[native] Add JSI bindings for LogInExistingDevice RPC
ClosedPublic
Actions

Authored by bartek on Mar 28 2024, 11:04 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Apr 21, 4:34 PM
Unknown Object (File)
Thu, Apr 11, 7:53 AM
Unknown Object (File)
Thu, Apr 11, 6:21 AM
Unknown Object (File)
Thu, Apr 11, 5:38 AM
Unknown Object (File)
Wed, Apr 10, 7:28 PM
Unknown Object (File)
Tue, Apr 9, 7:33 PM
Unknown Object (File)
Mon, Apr 8, 5:22 PM
Unknown Object (File)
Mon, Apr 8, 12:10 AM
View All 20 Files
Subscribers
ashoat
tomek

Details

Reviewers
varun
Commits
rCOMM70ffc9f4131d: [native] Add JSI bindings for LogInExistingDevice RPC
Summary

Exposed the LogInExistingDevice to JS - added it to CommRustModule

Depends on D11469

Test Plan

Called the RPC from JS on native. Repeated test plan from D11466.
Additionally, triggering commCoreModule.setCommServicesAuthMetadata() with the new token is able to refresh user session

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

bartek created this revision.Mar 28 2024, 11:04 PM
bartek held this revision as a draft.
Herald added subscribers: tomek, ashoat. · View Herald TranscriptMar 28 2024, 11:04 PM
bartek updated this revision to Diff 38531.Mar 28 2024, 11:07 PM

Rebase

Harbormaster failed remote builds in B27812: Diff 38529!Mar 28 2024, 11:21 PM
Harbormaster failed remote builds in B27814: Diff 38531!Mar 28 2024, 11:32 PM
bartek updated this revision to Diff 38533.Mar 28 2024, 11:56 PM

Fix typo

Harbormaster completed remote builds in B27816: Diff 38533.Mar 29 2024, 12:11 AM
bartek published this revision for review.Mar 29 2024, 12:19 AM
bartek added inline comments.
native/cpp/CommonCpp/NativeModules/CommRustModule.cpp
593 ↗(On Diff #38533)

A leftover dead code, the actual return createPromiseAsJSIValue is called at L567

varun accepted this revision.Mon, Apr 1, 9:35 PM
This revision is now accepted and ready to land.Mon, Apr 1, 9:35 PM
bartek updated this revision to Diff 38704.Wed, Apr 3, 2:53 AM

Rebase before landing

Harbormaster completed remote builds in B27932: Diff 38704.Wed, Apr 3, 3:11 AM
Closed by commit rCOMM70ffc9f4131d: [native] Add JSI bindings for LogInExistingDevice RPC (authored by bartek). · Explain WhyWed, Apr 3, 3:17 AM
This revision was automatically updated to reflect the committed changes.
bartek added a commit: rCOMM70ffc9f4131d: [native] Add JSI bindings for LogInExistingDevice RPC.

Revision Contents
Changeset List

PathSize
native/
cpp/
CommonCpp/
NativeModules/
5 lines
28 lines
_generated/
schema/
5 lines

Diff 38713

View Options

native/cpp/CommonCpp/NativeModules/CommRustModule.h

Loading...
View Options

native/cpp/CommonCpp/NativeModules/CommRustModule.cpp

Loading...
View Options

native/cpp/CommonCpp/_generated/rustJSI-generated.cpp

Loading...
View Options

native/cpp/CommonCpp/_generated/rustJSI.h

Loading...
View Options

native/schema/CommRustModuleSchema.js

Loading...