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

Authored by bartek on Wed, May 24, 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

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

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

bartek held this revision as a draft.

Herald added subscribers: atul, tomek. · View Herald TranscriptWed, May 24, 10:52 AM

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

Committed missing line

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

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

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

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

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

Closed by commit rCOMMad92458e922f: [web] Add missing thumbhash to newly created media message (authored by bartek). · Explain WhyWed, May 24, 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 27025

View Options

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

[web] Add missing thumbhash to newly created media messageClosedPublicActions