[web] Add missing thumbhash to newly created media message
ClosedPublic
Actions

Authored by bartek on May 24 2023, 10:52 AM.

Details

Reviewers

Commits

rCOMMad92458e922f: [web] Add missing thumbhash to newly created media message

Summary

This addresses https://phab.comm.dev/D7902#inline-52026.
I missed this change when implementing thumbhash generation on web.

Test Plan

Ensured with redux-devtools that the just created media message has thumbhash field set.

Diff Detail

Repository

rCOMM Comm

Branch

barthap/fix-thumbhash

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

bartek created this revision.May 24 2023, 10:52 AM

bartek held this revision as a draft.

Herald added subscribers: atul, tomek. · View Herald TranscriptMay 24 2023, 10:52 AM

bartek mentioned this in D7902: [web] Display thumbhash in chat.May 24 2023, 10:56 AM

Committed missing line

bartek published this revision for review.May 24 2023, 10:58 AM

Harbormaster completed remote builds in B19750: Diff 27010.May 24 2023, 11:10 AM

Harbormaster completed remote builds in B19751: Diff 27011.May 24 2023, 11:16 AM

ashoat accepted this revision.May 24 2023, 11:18 AM

This revision is now accepted and ready to land.May 24 2023, 11:18 AM

Closed by commit rCOMMad92458e922f: [web] Add missing thumbhash to newly created media message (authored by bartek). · Explain WhyMay 24 2023, 11:15 PM

This revision was automatically updated to reflect the committed changes.

bartek added a commit: rCOMMad92458e922f: [web] Add missing thumbhash to newly created media message.

Revision Contents
Changeset List

Path

Size

web/

input/

input-state-container.react.js

19 lines

Diff 27011

View Options

web/input/input-state-container.react.js

[web] Add missing thumbhash to newly created media messageClosedPublicActions