Page MenuHomePhabricator

Implement C++ code to update thread unread status
ClosedPublic

Authored by marcin on Jun 7 2022, 7:01 AM.
Tags
None
Referenced Files
F3357770: D4235.diff
Sun, Nov 24, 1:28 AM
Unknown Object (File)
Sat, Nov 2, 1:51 PM
Unknown Object (File)
Sat, Nov 2, 1:51 PM
Unknown Object (File)
Sat, Nov 2, 1:51 PM
Unknown Object (File)
Sat, Nov 2, 1:51 PM
Unknown Object (File)
Sat, Nov 2, 1:51 PM
Unknown Object (File)
Sat, Nov 2, 1:51 PM
Unknown Object (File)
Sat, Nov 2, 1:50 PM

Details

Summary

Implement C++ code to update thread unread status in SQLite.

Test Plan

No tsting plan. Further diffs will use this functionality onn iOS and Android and provide appropriate testing plan.

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

marcin requested review of this revision.Jun 7 2022, 7:31 AM
native/android/app/CMakeLists.txt
85–86 ↗(On Diff #13391)

Do we need both of these lines? Can we simply GLOB_RECURSE the COMMON_NATIVE_CODE line? Then I think we could potentially remove the GRPC_CODE line

native/android/app/CMakeLists.txt
85–86 ↗(On Diff #13391)

We can get rid of COMMON_NATIVE_INTERNAL by replacing GLOB with GLOB_RECURSE in COMMON_NATIVE_CODE. Perhaps we could also get rid of GRPC_CODE, but I would not do so. Grpc generated files end with .cc, so we would need to pass another globbing pattern to GLOB_RECURSE. Moreover it is in separate directory on the same directory level as NativeModules, so perhaps it is more readable to keep it separately.

Use one recursive globbing for NativeModules sources on Android.

tomek added a reviewer: ashoat.

Adding @ashoat because he gave a comment about build script

Rebase to master after CI change

This revision is now accepted and ready to land.Jun 8 2022, 8:05 AM