HomePhabricator
Diffusion Comm e9036e146415

Turn migration 38 into a no-op

Description

Turn migration 38 into a no-op

Summary:
This differential addresses concerns outlined in the following Linear issue:
https://linear.app/comm/issue/ENG-7638/investigate-android-crash-on-sqlite-migration-38

Test Plan:

  1. Revert "Introduce fallback mechanism for notifications decryption if MMKV initialization hasn't been executed yet." and "Implement notifications sessions storage

on top of MMKV. Migrate existing notifs account to SQLite" diffs.

  1. Build Android and iOS app and log in. this leaves the app in legacy state with notifications session in a single flat file.
  2. Reset reverts. Add logging to NotificationsCryptoModule informing that legacy notifications session storage was used to decrypt a notif.
  3. Build the app again. Send two notifs.
  4. Ensure that the first notif results in a log but the second doesn't (session was succesfully migrated to new place).

Reviewers: tomek, bartek, kamil

Reviewed By: tomek

Subscribers: ashoat

Differential Revision: https://phab.comm.dev/D11571

Details

Provenance
marcinAuthored on Apr 8 2024, 5:10 AM
Reviewer
tomek
Differential Revision
D11571: Turn migration 38 into a no-op
Parents
rCOMMbff93822188e: [lib] Introduce `FarcasterRelationshipRequest`, validators, and unit tests
Branches
Unknown
Tags
Unknown