Page MenuHomePhabricator

[lib][web] Add worker logic for adding search messages
ClosedPublic

Authored by inka on Jul 19 2024, 3:17 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Nov 1, 11:26 AM
Unknown Object (File)
Fri, Oct 18, 12:45 AM
Unknown Object (File)
Mon, Oct 14, 4:11 PM
Unknown Object (File)
Mon, Oct 14, 4:11 PM
Unknown Object (File)
Mon, Oct 14, 4:11 PM
Unknown Object (File)
Mon, Oct 14, 4:11 PM
Unknown Object (File)
Mon, Oct 14, 4:11 PM
Unknown Object (File)
Mon, Oct 14, 4:11 PM
Subscribers

Details

Summary

issue: ENG-8645

Test Plan

Added an update_search_messages operation in processDBStoreOperations in redux-utils.js. Tested that updateMessageSearchIndex was run and no errors showed up in the web console and worker console

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

inka requested review of this revision.Jul 19 2024, 3:33 AM
lib/message-search-types.js
5 ↗(On Diff #42520)

This should also be read-only

kamil added inline comments.
lib/message-search-types.js
9 ↗(On Diff #42520)

this one is not needed - we don't do any additional parsing, so you can use MessageSearchStoreOperation everywhere but if you want to keep it to make naming in ClientDBStoreOperations this is also fine

This revision is now accepted and ready to land.Jul 19 2024, 5:20 AM
lib/message-search-types.js
9 ↗(On Diff #42520)

I would rather keep it for consistency