Page MenuHomePhabricator

[NativeRustModule] implement uploading prekeys
ClosedPublic

Authored by kamil on Jan 12 2024, 4:40 AM.
Tags
None
Referenced Files
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
Unknown Object (File)
Fri, Sep 27, 9:30 PM
Subscribers

Details

Summary
Test Plan

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

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

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