Page MenuHomePhabricator
Differential D7423

[native] move emoji avatar creation from profile directory to components directory
ClosedPublic
Actions

Authored by ginsu on Apr 13 2023, 8:23 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Mar 15, 3:21 PM
Unknown Object (File)
Wed, Mar 12, 5:00 AM
Unknown Object (File)
Sat, Mar 8, 1:08 AM
Unknown Object (File)
Tue, Mar 4, 2:51 AM
Unknown Object (File)
Tue, Mar 4, 2:51 AM
Unknown Object (File)
Tue, Mar 4, 2:51 AM
Unknown Object (File)
Tue, Mar 4, 2:51 AM
Unknown Object (File)
Mon, Mar 3, 7:05 AM
View All 100 Files
Subscribers
ashoat
atul
tomek

Details

Reviewers
ashoat
atul
Commits
rCOMM94c79f1808ee: [native] move emoji avatar creation from profile directory to components…
Summary

This is a move diff, I want to move this component to components directory since both profile and chat will be using this. Probably should have done this from the get go

Depends on D7378

Test Plan

flow

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

ginsu created this revision.Apr 13 2023, 8:23 AM
Herald added subscribers: atul, tomek, ashoat. · View Herald TranscriptApr 13 2023, 8:23 AM
ginsu edited the summary of this revision. (Show Details)Apr 13 2023, 8:24 AM
ginsu added reviewers: ashoat, atul.
ginsu added a parent revision: D7378: [native] add edit avatar badge to thread avatar.
ginsu edited the summary of this revision. (Show Details)Apr 13 2023, 8:27 AM
Harbormaster completed remote builds in B18330: Diff 25114.Apr 13 2023, 8:41 AM
ginsu requested review of this revision.Apr 13 2023, 8:41 AM
ginsu added a child revision: D7424: [native] introduce useSaveUserAvatar hook.Apr 13 2023, 8:44 AM
ashoat accepted this revision.Apr 13 2023, 9:01 AM

Does nothing actually use this component? Usually we'd have to update some import paths as well

This revision is now accepted and ready to land.Apr 13 2023, 9:01 AM
ginsu updated this revision to Diff 25123.Apr 13 2023, 9:13 AM

rebase before landing

ginsu added a comment.EditedApr 13 2023, 9:22 AM

Does nothing actually use this component? Usually we'd have to update some import paths as well

This will be addressed in D7324 but going to update this after EmojiAvatarCreation handles thread avatars and then I will also add the imports for the chat navigator as well, but to answer your question at this point in the stack nothing does call and use this component

Harbormaster completed remote builds in B18336: Diff 25123.Apr 13 2023, 9:31 AM
Closed by commit rCOMM94c79f1808ee: [native] move emoji avatar creation from profile directory to components… (authored by ginsu). · Explain WhyApr 13 2023, 9:37 AM
This revision was automatically updated to reflect the committed changes.
ginsu added a commit: rCOMM94c79f1808ee: [native] move emoji avatar creation from profile directory to components….

Revision Contents
Changeset List

PathSize
native/
components/
profile/
profile/

Diff 25127

View Options

native/components/emoji-avatar-creation.react.js

Show First 20 Lines • Show All 191 Lines • ▼ Show 20 Lines if (messagesFromDatabase.size() > 0) {
// if there are undelivered messages in the database. If so, we are // if there are undelivered messages in the database. If so, we are
// erasing the messages to deliver from rabbitMQ which are handled by // erasing the messages to deliver from rabbitMQ which are handled by
// DeliveryBroker. // DeliveryBroker.
DeliveryBroker::getInstance().erase(clientDeviceID); DeliveryBroker::getInstance().erase(clientDeviceID);
} }
tunnelbroker::GetResponse response; tunnelbroker::GetResponse response;
auto respondToWriter = auto respondToWriter =
[&writer, &response](std::string fromDeviceID, std::string payload) { [&writer, &response](std::string fromDeviceID, std::string payload) {
response.set_fromdeviceid(fromDeviceID); response.mutable_responsemessage()->set_fromdeviceid(fromDeviceID);
response.set_payload(payload); response.mutable_responsemessage()->set_payload(payload);
if (!writer->Write(response)) { if (!writer->Write(response)) {
throw std::runtime_error( throw std::runtime_error(
"gRPC: 'Get' writer error on sending data to the client"); "gRPC: 'Get' writer error on sending data to the client");
} }
response.Clear(); response.Clear();
}; };
for (auto &messageFromDatabase : messagesFromDatabase) { for (auto &messageFromDatabase : messagesFromDatabase) {
respondToWriter( respondToWriter(
Show All 28 Lines
View Options

native/profile/emoji-avatar-creation.react.js

Show First 20 Lines • Show All 191 Lines • ▼ Show 20 Lines if (messagesFromDatabase.size() > 0) {
// if there are undelivered messages in the database. If so, we are // if there are undelivered messages in the database. If so, we are
// erasing the messages to deliver from rabbitMQ which are handled by // erasing the messages to deliver from rabbitMQ which are handled by
// DeliveryBroker. // DeliveryBroker.
DeliveryBroker::getInstance().erase(clientDeviceID); DeliveryBroker::getInstance().erase(clientDeviceID);
} }
tunnelbroker::GetResponse response; tunnelbroker::GetResponse response;
auto respondToWriter = auto respondToWriter =
[&writer, &response](std::string fromDeviceID, std::string payload) { [&writer, &response](std::string fromDeviceID, std::string payload) {
response.set_fromdeviceid(fromDeviceID); response.mutable_responsemessage()->set_fromdeviceid(fromDeviceID);
response.set_payload(payload); response.mutable_responsemessage()->set_payload(payload);
if (!writer->Write(response)) { if (!writer->Write(response)) {
throw std::runtime_error( throw std::runtime_error(
"gRPC: 'Get' writer error on sending data to the client"); "gRPC: 'Get' writer error on sending data to the client");
} }
response.Clear(); response.Clear();
}; };
for (auto &messageFromDatabase : messagesFromDatabase) { for (auto &messageFromDatabase : messagesFromDatabase) {
respondToWriter( respondToWriter(
Show All 28 Lines