Page MenuHomePhabricator

[native] getNotificationsOneTimeKeys
ClosedPublic

Authored by varun on Sep 5 2023, 7:32 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Mar 16, 11:15 PM
Unknown Object (File)
Sun, Mar 16, 11:15 PM
Unknown Object (File)
Sun, Mar 16, 11:14 PM
Unknown Object (File)
Wed, Mar 12, 10:21 PM
Unknown Object (File)
Fri, Mar 7, 10:15 PM
Unknown Object (File)
Tue, Mar 4, 7:34 AM
Unknown Object (File)
Tue, Mar 4, 7:27 AM
Unknown Object (File)
Feb 20 2025, 9:05 AM

Details

Summary

we need to send the identity service notif one-time keys

Depends on D9084

Test Plan

console logged the one-time keys in account-selectors.js
tested more in subsequent diff when register_user is called

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

varun requested review of this revision.Sep 5 2023, 7:50 AM
atul added a subscriber: bartek.
atul added inline comments.
native/cpp/CommonCpp/Notifications/BackgroundDataStorage/NotificationsCryptoModule.cpp
209–213 ↗(On Diff #30775)

Don't understand what's going on here, would want @marcin or @bartek to take a closer look

native/cpp/CommonCpp/Notifications/BackgroundDataStorage/NotificationsCryptoModule.h
36 ↗(On Diff #30775)

I'd prefer something like numberOfKeys but feel free to go with whatever

This revision is now accepted and ready to land.Sep 6 2023, 2:07 PM
This revision was automatically updated to reflect the committed changes.