Page MenuHomePhabricator
Differential D5060

[native] Update `thumbnailID` upon successful thumbnail upload
ClosedPublic
Actions

Authored by atul on Sep 5 2022, 9:18 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Nov 11, 2:51 AM
Unknown Object (File)
Tue, Nov 5, 11:28 AM
Unknown Object (File)
Sat, Nov 2, 1:14 PM
Unknown Object (File)
Sat, Nov 2, 1:14 PM
Unknown Object (File)
Sat, Nov 2, 1:14 PM
Unknown Object (File)
Sat, Nov 2, 1:14 PM
Unknown Object (File)
Sat, Nov 2, 1:14 PM
Unknown Object (File)
Sat, Nov 2, 1:11 PM
View All Files
Subscribers
ashoat

Details

Reviewers
• abosh
tomek
marcin
Commits
rCOMMa7ba71744ae8: Revert "[lib] Update `SendMultimediaMessageRequest` type"
rCOMM7ed99628ce2f: [native] Update `thumbnailID` upon successful thumbnail upload
Summary

Need to include the thumbnailID in addition to the thumbnailURL in the updateMediaPayload.

Depends on D5059

Test Plan

sendMultimediaMessage in subsequent diff gets correctly constructed mediaMessageContents with expected values.

Screen Shot 2022-09-06 at 12.26.19 AM.png (692×2 px, 855 KB)

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

atul created this revision.Sep 5 2022, 9:18 PM
Herald added a subscriber: ashoat. · View Herald TranscriptSep 5 2022, 9:18 PM
Harbormaster completed remote builds in B11898: Diff 16341.Sep 5 2022, 9:27 PM
atul requested review of this revision.Sep 5 2022, 9:27 PM
atul added a child revision: D5061: [lib] Update `SendMultimediaMessageRequest` type.Sep 5 2022, 9:42 PM
atul edited the test plan for this revision. (Show Details)Sep 5 2022, 9:43 PM
atul updated this revision to Diff 16389.Sep 6 2022, 6:09 PM

ignore

atul updated this revision to Diff 16390.Sep 6 2022, 6:11 PM

ignore

Harbormaster completed remote builds in B11938: Diff 16389.Sep 6 2022, 6:19 PM
Harbormaster completed remote builds in B11939: Diff 16390.Sep 6 2022, 6:22 PM
atul removed a reviewer: • jacek.Sep 7 2022, 5:34 PM
atul added 1 blocking reviewer(s): tomek.Sep 7 2022, 10:22 PM
tomek accepted this revision.Sep 8 2022, 8:07 AM
This revision is now accepted and ready to land.Sep 8 2022, 8:07 AM
atul updated this revision to Diff 16519.Sep 8 2022, 1:56 PM

rebase before landing

This revision was landed with ongoing or failed builds.Sep 8 2022, 1:57 PM
Closed by commit rCOMM7ed99628ce2f: [native] Update `thumbnailID` upon successful thumbnail upload (authored by atul). · Explain Why
This revision was automatically updated to reflect the committed changes.
atul added a commit: rCOMM7ed99628ce2f: [native] Update `thumbnailID` upon successful thumbnail upload.
Harbormaster completed remote builds in B12024: Diff 16519.Sep 8 2022, 2:06 PM
atul added a commit: rCOMMa7ba71744ae8: Revert "[lib] Update `SendMultimediaMessageRequest` type".Sep 9 2022, 2:28 AM

Revision Contents
Changeset List

PathSize
native/
input/
3 lines

Diff 16390

View Options

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

Loading...