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)
Tue, Apr 1, 8:17 AM
Unknown Object (File)
Wed, Mar 19, 2:12 PM
Unknown Object (File)
Thu, Mar 13, 7:39 AM
Unknown Object (File)
Wed, Mar 12, 5:39 AM
Unknown Object (File)
Tue, Mar 11, 3:38 PM
Unknown Object (File)
Mar 8 2025, 1:05 AM
Unknown Object (File)
Mar 8 2025, 1:05 AM
Unknown Object (File)
Mar 8 2025, 1:05 AM
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 16389

View Options

blah

FROM commapp/services-base:1.2 FROM commapp/services-base:1.3
RUN apt-get update && \ RUN apt-get update && \
apt-get install -y uuid-dev && \ apt-get install -y uuid-dev && \
rm -rf /var/lib/apt/lists/* rm -rf /var/lib/apt/lists/*
ARG COMM_TEST_SERVICES ARG COMM_TEST_SERVICES
ARG COMM_SERVICES_SANDBOX ARG COMM_SERVICES_SANDBOX
ENV COMM_TEST_SERVICES=${COMM_TEST_SERVICES} ENV COMM_TEST_SERVICES=${COMM_TEST_SERVICES}
ENV COMM_SERVICES_SANDBOX=${COMM_SERVICES_SANDBOX} ENV COMM_SERVICES_SANDBOX=${COMM_SERVICES_SANDBOX}
WORKDIR /transferred WORKDIR /transferred
COPY native/cpp/CommonCpp/grpc/protos/blob.proto protos/blob.proto
COPY services/lib/cmake-components cmake-components
COPY services/lib/docker/ scripts/ COPY services/lib/docker/ scripts/
COPY services/blob/ . COPY services/blob/ blob
COPY services/lib/src/* src/ COPY services/lib/src lib/src/
RUN scripts/build_service.sh WORKDIR /transferred/blob
ADD native/cpp/CommonCpp/grpc grpc
RUN ../scripts/build_service.sh
WORKDIR /transferred
CMD if [ "$COMM_TEST_SERVICES" -eq 1 ]; then scripts/run_tests.sh; else scripts/run_service.sh; fi CMD if [ "$COMM_TEST_SERVICES" -eq 1 ]; then scripts/run_tests.sh; else scripts/run_service.sh; fi
View Options

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

FROM commapp/services-base:1.2 FROM commapp/services-base:1.3
RUN apt-get update && \ RUN apt-get update && \
apt-get install -y uuid-dev && \ apt-get install -y uuid-dev && \
rm -rf /var/lib/apt/lists/* rm -rf /var/lib/apt/lists/*
ARG COMM_TEST_SERVICES ARG COMM_TEST_SERVICES
ARG COMM_SERVICES_SANDBOX ARG COMM_SERVICES_SANDBOX
ENV COMM_TEST_SERVICES=${COMM_TEST_SERVICES} ENV COMM_TEST_SERVICES=${COMM_TEST_SERVICES}
ENV COMM_SERVICES_SANDBOX=${COMM_SERVICES_SANDBOX} ENV COMM_SERVICES_SANDBOX=${COMM_SERVICES_SANDBOX}
WORKDIR /transferred WORKDIR /transferred
COPY native/cpp/CommonCpp/grpc/protos/blob.proto protos/blob.proto
COPY services/lib/cmake-components cmake-components
COPY services/lib/docker/ scripts/ COPY services/lib/docker/ scripts/
COPY services/blob/ . COPY services/blob/ blob
COPY services/lib/src/* src/ COPY services/lib/src lib/src/
RUN scripts/build_service.sh WORKDIR /transferred/blob
ADD native/cpp/CommonCpp/grpc grpc
RUN ../scripts/build_service.sh
WORKDIR /transferred
CMD if [ "$COMM_TEST_SERVICES" -eq 1 ]; then scripts/run_tests.sh; else scripts/run_service.sh; fi CMD if [ "$COMM_TEST_SERVICES" -eq 1 ]; then scripts/run_tests.sh; else scripts/run_service.sh; fi