[web] Introduce `WebEditThreadAvatarProvider`
ClosedPublic
Actions

Authored by atul on Aug 13 2023, 8:44 PM.

Details

Reviewers

ashoat
ginsu

Commits

rCOMM70c34b4ab11c: [web] Introduce `WebEditThreadAvatarProvider`

Summary

Platform-specific BaseEditThreadAvatarProvider wrapper that passes in activeThreadID.

Test Plan

console.log value of activeThreadID and ensure that it's correct.

Diff Detail

Repository

rCOMM Comm

Branch

arcpatch-D8800 (branched from master)

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

atul created this revision.Aug 13 2023, 8:44 PM

Herald added a subscriber: tomek. · View Herald TranscriptAug 13 2023, 8:44 PM

Harbormaster completed remote builds in B21722: Diff 29849.Aug 13 2023, 9:02 PM

atul requested review of this revision.Aug 13 2023, 9:02 PM

atul added a child revision: D8801: [web] Add `showSpinner` prop to `ThreadAvatar`.Aug 14 2023, 12:05 AM

ginsu accepted this revision.Aug 14 2023, 9:18 AM

This revision is now accepted and ready to land.Aug 14 2023, 9:18 AM

rebase and land

This revision was landed with ongoing or failed builds.Aug 14 2023, 10:30 PM

Closed by commit rCOMM70c34b4ab11c: [web] Introduce `WebEditThreadAvatarProvider` (authored by atul). · Explain Why

This revision was automatically updated to reflect the committed changes.

atul added a commit: rCOMM70c34b4ab11c: [web] Introduce `WebEditThreadAvatarProvider`.

Revision Contents
Changeset List

Path

Size

web/

app.react.js

11 lines

avatars/

native-edit-thread-avatar-provider.react.js

25 lines

Diff 29874

View Options

web/app.react.js