This differential implements encryption with notifications session on native
Details
Details
Diff Detail
Diff Detail
- Repository
- rCOMM Comm
- Branch
- marcin/eng-8237
- Lint
No Lint Coverage - Unit
No Test Coverage
Event Timeline
| native/push/encrypted-notif-utils-api.js | ||
|---|---|---|
| 21 | Isn't this conversion equivalent? | |
| native/push/encrypted-notif-utils-api.js | ||
|---|---|---|
| 21 | Type definition at line 13 expects literal union of 1 | 0 while toString() can yield any string. Flow complains when this change is applied. | |
| native/cpp/CommonCpp/NativeModules/CommCoreModule.cpp | ||
|---|---|---|
| 1577–1584 | same comment as here: https://phab.comm.dev/D12668#inline-74094 | |
| native/cpp/CommonCpp/NativeModules/CommCoreModule.h | ||
| 141 | is it change accidental? If not please update .cpp file and Schema in .js file | |
| native/cpp/CommonCpp/Notifications/BackgroundDataStorage/NotificationsCryptoModule.h | ||
| 120 | in CommCoreModuleSchema it makes sense to keep the suffix but here because it's NotificationsCryptoModule it seems redundant | |
| native/push/encrypted-notif-utils-api.js | ||
| 17 | I prefer to add encryptNotification to sqliteAPI and use it here (and on web), but up to you | |
| native/push/encrypted-notif-utils-api.js | ||
|---|---|---|
| 17 | olmAPI* - sorry for confusing | |
| native/push/encrypted-notif-utils-api.js | ||
|---|---|---|
| 17 | Added in next diff | |