Page MenuHomePhabricator

[web] Introduce getRelatedMessages to shared worker
ClosedPublic

Authored by ashoat on Jul 14 2024, 6:11 PM.
Tags
None
Referenced Files
F3181559: D12738.id42368.diff
Fri, Nov 8, 6:29 AM
F3181298: D12738.id42271.diff
Fri, Nov 8, 6:07 AM
F3180691: D12738.id42316.diff
Fri, Nov 8, 5:09 AM
Unknown Object (File)
Thu, Oct 24, 9:14 AM
Unknown Object (File)
Tue, Oct 22, 12:09 PM
Unknown Object (File)
Tue, Oct 22, 11:05 AM
Unknown Object (File)
Fri, Oct 18, 10:29 PM
Unknown Object (File)
Fri, Oct 18, 10:29 PM
Subscribers
None

Details

Summary

This diff introduces the code on web to interface with our new getRelatedMessages API exposed via WASM.

Depends on D12737

Test Plan
  1. In combination with later diffs, I tested a patch on both native and web that hit getRelatedMessages with a specific message ID and confirmed that it returned the message
  2. I tested a message ID that did not exist and confirmed I got undefined back

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

This revision is now accepted and ready to land.Jul 15 2024, 2:53 AM
ashoat retitled this revision from [web] Introduce getLatestMessageEdit to shared worker to [web] Introduce getRelatedMessages to shared worker.Jul 15 2024, 7:02 PM
ashoat edited the summary of this revision. (Show Details)
ashoat edited the test plan for this revision. (Show Details)
ashoat retitled this revision from [web] Introduce getRelatedMessages to shared worker to [web] Introduce getRelatedMessages to shared worker.
ashoat edited the test plan for this revision. (Show Details)
ashoat edited the summary of this revision. (Show Details)

Rework into getRelatedMessages, which can be reused for rendering search results