Paths

Table of Contentst

[web] introduce message searching to shared worker
ClosedPublic
Actions

Authored by kamil on Jul 23 2024, 3:05 AM.

Details

Reviewers

tomek
inka

Commits

rCOMM81fc7466f203: [web] introduce message searching to shared worker

Summary

The code on the web to interface with the search API exposed via WASM.

Depends on D12851

Test Plan

Tested later in the stack by @inka.

Diff Detail

Repository

rCOMM Comm

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

kamil created this revision.Jul 23 2024, 3:05 AM

kamil held this revision as a draft.

Herald added a subscriber: ashoat. · View Herald TranscriptJul 23 2024, 3:05 AM

kamil added a child revision: D12853: [native] introduce message searching to `CommCoreModule`.Jul 23 2024, 3:09 AM

kamil edited the summary of this revision. (Show Details)Jul 23 2024, 3:27 AM

Harbormaster completed remote builds in B30597: Diff 42671.Jul 23 2024, 3:37 AM

kamil published this revision for review.Jul 23 2024, 5:03 AM

kamil added inline comments.

web/types/worker-types.js
298–306	we can merge those two and name something like `MessagesResponse` but not sure if worth

tomek accepted this revision.Jul 23 2024, 6:28 AM

tomek added inline comments.

web/types/worker-types.js
298–306	Sounds quite cheap, we can have e.g. `GetMessagesResponse`.

This revision is now accepted and ready to land.Jul 23 2024, 6:28 AM

merge types

Harbormaster completed remote builds in B30616: Diff 42703.Jul 23 2024, 8:14 AM

Closed by commit rCOMM81fc7466f203: [web] introduce message searching to shared worker (authored by kamil). · Explain WhyJul 23 2024, 8:18 AM

This revision was automatically updated to reflect the committed changes.

kamil added a commit: rCOMM81fc7466f203: [web] introduce message searching to shared worker.

Revision Contents
Changeset List

Path

Size

web/

shared-worker/

worker/

shared-worker.js

11 lines

types/

worker-types.js

21 lines

Diff 42671

View Options

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