Differential D13030

[native] introduce resetting Outbound P2P Messages to `CommCoreModule`
ClosedPublic
Actions

Authored by kamil on Aug 8 2024, 4:26 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sat, Sep 7, 1:47 PM

	Unknown Object (File)
	Sat, Sep 7, 11:37 AM

	Unknown Object (File)
	Sat, Sep 7, 11:36 AM

	Unknown Object (File)
	Sat, Sep 7, 6:20 AM

	Unknown Object (File)
	Thu, Sep 5, 1:30 AM

	Unknown Object (File)
	Tue, Sep 3, 2:59 AM

	Unknown Object (File)
	Mon, Sep 2, 3:49 PM

	Unknown Object (File)
	Sun, Sep 1, 11:59 PM

View All 42 Files

Subscribers

Details

Reviewers

Commits

rCOMM12da08a568d6: [native] introduce resetting Outbound P2P Messages to `CommCoreModule`

Summary

The code on native to interface with search API using JSI.

Depends on D13029

Test Plan

Tested in D13032

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

kamil created this revision.Aug 8 2024, 4:26 AM

kamil held this revision as a draft.

Herald added a subscriber: ashoat. · View Herald TranscriptAug 8 2024, 4:26 AM

Harbormaster failed remote builds in B31009: Diff 43258!Aug 8 2024, 4:49 AM

kamil added a child revision: D13031: [lib] introduce resetting Outbound P2P Messages to `SQLiteAPI`.Aug 8 2024, 5:07 AM

kamil published this revision for review.Aug 8 2024, 5:30 AM

kamil edited the test plan for this revision. (Show Details)

tomek accepted this revision.Mon, Aug 12, 1:17 AM

tomek added inline comments.

native/cpp/CommonCpp/NativeModules/CommCoreModule.cpp
2885–2892 ↗	(On Diff #43258)	We're doing something similar in a lot of places. Might be a good idea to introduce some utils for std collections <-> jsi collections conversions. (But we shouldn't prioritize it soon)

This revision is now accepted and ready to land.Mon, Aug 12, 1:17 AM

add transaction

kamil added inline comments.Tue, Aug 13, 3:50 AM

native/cpp/CommonCpp/NativeModules/CommCoreModule.cpp
2885–2892 ↗	(On Diff #43258)	ENG-9033

Harbormaster completed remote builds in B31056: Diff 43327.Tue, Aug 13, 3:57 AM

Closed by commit rCOMM12da08a568d6: [native] introduce resetting Outbound P2P Messages to `CommCoreModule` (authored by kamil). · Explain WhyTue, Aug 13, 4:10 AM

This revision was automatically updated to reflect the committed changes.

kamil added a commit: rCOMM12da08a568d6: [native] introduce resetting Outbound P2P Messages to `CommCoreModule`.

Revision Contents
Changeset List

Path

Size

native/

cpp/

CommonCpp/

NativeModules/

CommCoreModule.h

4 lines

CommCoreModule.cpp

48 lines

_generated/

commJSI-generated.cpp

schema/

CommCoreModuleSchema.js

3 lines

Diff 43330

native/cpp/CommonCpp/NativeModules/CommCoreModule.h

Loading...

native/cpp/CommonCpp/NativeModules/CommCoreModule.cpp

Loading...

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

Loading...

native/cpp/CommonCpp/_generated/commJSI.h

Loading...

native/schema/CommCoreModuleSchema.js

Loading...