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
F3357758: D12801.diff
Sun, Nov 24, 1:23 AM
Unknown Object (File)
Mon, Nov 4, 12:55 PM
Unknown Object (File)
Mon, Nov 4, 12:48 PM
Unknown Object (File)
Fri, Nov 1, 11:26 AM
Unknown Object (File)
Oct 18 2024, 12:45 AM
Unknown Object (File)
Oct 14 2024, 4:11 PM
Unknown Object (File)
Oct 14 2024, 4:11 PM
Unknown Object (File)
Oct 14 2024, 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
Branch
inka/search
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

This should also be read-only

kamil added inline comments.
lib/message-search-types.js
9

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

I would rather keep it for consistency