Page MenuHomePhabricator
Differential D9147

[lib/web/native] Improve conversion utils
ClosedPublic
Actions

Authored by michal on Sep 12 2023, 5:20 AM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Jul 10, 12:18 AM
Unknown Object (File)
Wed, Jul 10, 12:18 AM
Unknown Object (File)
Wed, Jul 10, 12:18 AM
Unknown Object (File)
Wed, Jul 10, 12:09 AM
Unknown Object (File)
Tue, Jul 9, 10:33 AM
Unknown Object (File)
Fri, Jul 5, 1:32 PM
Unknown Object (File)
Fri, Jul 5, 4:41 AM
Unknown Object (File)
Fri, Jun 28, 12:00 PM
View All 42 Files
Subscribers
ashoat
tomek

Details

Reviewers
kamil
inka
atul
Commits
rCOMM375b9a2daeff: [lib/web/native] Improve conversion utils
Summary

Improve the conversion utils situation for the future diffs:

  1. Rename convertNotificationThreadIDToNewIDSchema to convertNonPendingIDToNewSchema and add idPrefix param instead of always using ashoats id
  2. Add convertIDToNewSchema and use it inside convertDraftKeyToNewIDSchema
Test Plan

yarn flow-all to test (1)
yarn jest-all to test (2) -> draft migration tests

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Revision Contents
Changeset List

PathSize
lib/
utils/
50 lines
native/
push/
8 lines
8 lines
9 lines
web/
push-notif/
9 lines
6 lines

Diff 31303

View Options

lib/utils/migration-utils.js

Loading...
View Options

native/push/android.js

Loading...
View Options

native/push/comm-ios-notification.js

Loading...
View Options

native/push/push-handler.react.js

Loading...
View Options

web/push-notif/push-notifs-handler.js

Loading...
View Options

web/push-notif/service-worker.js

Loading...