Page MenuHomePhabricator

[lib] allow missing `messageIDs` while comparing SQLite with `redux-persist`
ClosedPublic

Authored by kamil on Apr 25 2023, 10:04 AM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Apr 3, 4:17 PM
Unknown Object (File)
Wed, Apr 3, 4:17 PM
Unknown Object (File)
Wed, Apr 3, 4:13 PM
Unknown Object (File)
Feb 27 2024, 3:25 AM
Unknown Object (File)
Feb 23 2024, 12:27 PM
Unknown Object (File)
Feb 18 2024, 5:39 PM
Unknown Object (File)
Feb 18 2024, 4:59 PM
Unknown Object (File)
Feb 18 2024, 3:40 PM
Subscribers

Details

Summary

This allows threadID that is present in database but not redux. This will help while debugging - instead of an error that cannot read messageIDs of undefined we will have a more readable message, having information on which keys are different. Is it an issue for one key and e.g. it was not removed from DB while needed or there is some issue and redux-persist state was purged.

Depends on D7611

Test Plan

Make inconsistency and check if it is properly detected and logged

image.png (310×277 px, 57 KB)

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable