Page MenuHomePhabricator
Differential D12738

[web] Introduce getRelatedMessages to shared worker
ClosedPublic
Actions

Authored by ashoat on Jul 14 2024, 6:11 PM.
Tags
None
Referenced Files
F2719646: D12738.diff
Mon, Sep 16, 1:29 PM
F2715035: D12738.id42271.diff
Mon, Sep 16, 2:58 AM
Unknown Object (File)
Sun, Sep 15, 10:07 AM
Unknown Object (File)
Sun, Sep 15, 10:06 AM
Unknown Object (File)
Mon, Sep 9, 1:14 PM
Unknown Object (File)
Wed, Aug 28, 3:56 PM
Unknown Object (File)
Wed, Aug 28, 8:29 AM
Unknown Object (File)
Tue, Aug 27, 8:30 PM
View All 62 Files
Subscribers
None

Details

Reviewers
kamil
tomek
marcin
Commits
rCOMMba1a449ef488: [web] Introduce getRelatedMessages to shared worker
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

ashoat created this revision.Jul 14 2024, 6:11 PM
ashoat added a child revision: D12739: [native] Introduce getRelatedMessages to CommCoreModule.
Harbormaster completed remote builds in B30311: Diff 42271.Jul 14 2024, 6:28 PM
ashoat requested review of this revision.Jul 14 2024, 6:28 PM
marcin accepted this revision.Jul 15 2024, 2:53 AM
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)
ashoat updated this revision to Diff 42310.Jul 15 2024, 7:14 PM

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

Harbormaster failed remote builds in B30348: Diff 42310!Jul 15 2024, 7:26 PM
ashoat updated this revision to Diff 42316.Jul 15 2024, 7:55 PM

ESLint fix

Harbormaster failed remote builds in B30354: Diff 42316!Jul 15 2024, 8:26 PM
ashoat updated this revision to Diff 42333.Jul 16 2024, 4:08 AM

Rebase

Harbormaster completed remote builds in B30370: Diff 42333.Jul 16 2024, 4:24 AM
Closed by commit rCOMMba1a449ef488: [web] Introduce getRelatedMessages to shared worker (authored by ashoat). · Explain WhyJul 16 2024, 6:46 PM
This revision was automatically updated to reflect the committed changes.
ashoat added a commit: rCOMMba1a449ef488: [web] Introduce getRelatedMessages to shared worker.

Revision Contents
Changeset List

PathSize
web/
shared-worker/
worker/
9 lines
types/
19 lines

Diff 42333

View Options

web/shared-worker/worker/shared-worker.js

Loading...
View Options

web/types/worker-types.js

Loading...