Differential D10294
[lib][keyserver] Replace `legacyRawThreadInfoValidator` with `rawThreadInfoValidator` Authored by atul on Dec 11 2023, 11:58 AM. Tags None Referenced Files
Subscribers
Details To address keyserver validation issues during testing. NOTE: Had issue with threadStoreValidator but deferring for now since that's only relevant for web (called from initialReduxStateValidator) and we're focusing on native for now.
Depends on D10286
Diff Detail
Event TimelineComment Actions How come legacyRawThreadInfoValidator isn't removed here? Is it still needed somewhere? Comment Actions It isn't removed because it's part of rawThreadInfoValidator which is a union of legacyRawThreadInfoValidator and minimallyEncodedRawThreadInfoValidator. It's also used in keyserver/scripts/generate-converter-from-validator.js, which I think is a one-off script and doesn't need to be updated CC @michal Comment Actions
Yeah it was used to migrate generate migrations from validators. We won't need to generate any new migrations. |