Page MenuHomePhabricator

[NativeRustModule] implement uploading prekeys
ClosedPublic

Authored by kamil on Jan 12 2024, 4:40 AM.
Tags
None
Referenced Files
F2902323: D10618.id35673.diff
Sat, Oct 5, 4:58 PM
F2902316: D10618.id35672.diff
Sat, Oct 5, 4:55 PM
F2902311: D10618.id35659.diff
Sat, Oct 5, 4:53 PM
F2902309: D10618.id35618.diff
Sat, Oct 5, 4:53 PM
F2902282: D10618.id35570.diff
Sat, Oct 5, 4:40 PM
Unknown Object (File)
Wed, Oct 2, 10:15 AM
Unknown Object (File)
Mon, Sep 30, 2:33 AM
Unknown Object (File)
Sat, Sep 28, 9:09 PM
Subscribers

Details

Summary
Test Plan

Call this method and test if it works by inspecting staging DDB table

Diff Detail

Repository
rCOMM Comm
Branch
rust-promise-magaer
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

kamil held this revision as a draft.
kamil published this revision for review.Jan 15 2024, 2:43 AM
varun added inline comments.
native/native_rust_library/src/lib.rs
1130 ↗(On Diff #35618)

we shouldn't have made these field names plural in the .proto file. created this task: https://linear.app/comm/issue/ENG-6449/rename-new-content-prekeys-and-new-notif-prekeys-in-proto-file

This revision is now accepted and ready to land.Jan 15 2024, 12:37 PM