Page MenuHomePhabricator
Differential D3505

[web] [refactor] message preview styles solidified into one ternary statement
ClosedPublic
Actions

Authored by • benschac on Mar 23 2022, 12:20 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Mar 17, 9:59 AM
Unknown Object (File)
Wed, Mar 12, 5:12 AM
Unknown Object (File)
Fri, Mar 7, 5:00 AM
Unknown Object (File)
Feb 16 2025, 5:20 AM
Unknown Object (File)
Feb 10 2025, 6:41 PM
Unknown Object (File)
Feb 8 2025, 10:29 AM
Unknown Object (File)
Feb 3 2025, 11:36 PM
Unknown Object (File)
Feb 3 2025, 10:43 AM
View All Files
Subscribers
• adrian
ashoat
• karol
tomek

Details

Reviewers
atul
• jacek
tomek
ashoat
Commits
rCOMMec5704f78a30: [web] [refactor] message preview styles solidified into one ternary statement
Summary

move styles into a single call

Test Plan

click and hover about chat thread list items. Functionality should be the same

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

• benschac created this revision.Mar 23 2022, 12:20 PM
Herald added subscribers: • karol, • adrian, tomek, ashoat. · View Herald TranscriptMar 23 2022, 12:20 PM
Harbormaster completed remote builds in B7557: Diff 10628.Mar 23 2022, 12:25 PM
• benschac requested review of this revision.Mar 23 2022, 12:25 PM
• benschac added a parent revision: D3504: [web] [refactor] destructure props in message preview component.Mar 23 2022, 12:26 PM
• benschac added a child revision: D3506: [web] [refactor] simplify username logic in message preview.Mar 23 2022, 12:34 PM
• benschac updated this revision to Diff 10635.Mar 23 2022, 1:28 PM

remove array

Harbormaster completed remote builds in B7564: Diff 10635.Mar 23 2022, 1:33 PM
• benschac added reviewers: • jacek, tomek.Mar 23 2022, 1:54 PM
• benschac mentioned this in D3500: [web] [refactor] css color should be white.Mar 24 2022, 8:53 AM
• benschac updated this revision to Diff 10667.Mar 24 2022, 9:46 AM

rename classes from color to actions

Harbormaster completed remote builds in B7596: Diff 10667.Mar 24 2022, 9:51 AM
ashoat accepted this revision.Mar 24 2022, 6:04 PM
This revision is now accepted and ready to land.Mar 24 2022, 6:04 PM
tomek accepted this revision.Mar 25 2022, 3:06 AM
• benschac added a comment.Mar 25 2022, 6:54 AM

css['white'] project for this string before landing.

• benschac added a comment.Mar 25 2022, 10:01 AM
In D3505#95835, @benschac wrote:

css['white'] project for this string before landing.

searched the project and didn't see it anywhere.

• benschac updated this revision to Diff 10702.Mar 25 2022, 10:18 AM

rebase

This revision was landed with ongoing or failed builds.Mar 25 2022, 10:20 AM
Closed by commit rCOMMec5704f78a30: [web] [refactor] message preview styles solidified into one ternary statement (authored by • benschac). · Explain Why
This revision was automatically updated to reflect the committed changes.
• benschac added a commit: rCOMMec5704f78a30: [web] [refactor] message preview styles solidified into one ternary statement.
• benschac added a comment.Mar 25 2022, 10:22 AM

I should have waited for the CI to finish. Sorry everyone.

Harbormaster completed remote builds in B7621: Diff 10702.Mar 25 2022, 10:23 AM

Revision Contents
Changeset List

PathSize
web/
chat/
8 lines
8 lines

Diff 10703

View Options

web/chat/chat-thread-list.css

  • This file was copied to services/backup/src/Reactors/client/blob/BlobPutClientReactor.cpp.
#pragma once #pragma once
#include "ClientBidiReactorBase.h"
#include "Constants.h" #include "Constants.h"
#include "../_generated/blob.grpc.pb.h" #include "../_generated/blob.grpc.pb.h"
#include "../_generated/blob.pb.h" #include "../_generated/blob.pb.h"
#include "ClientBidiReactorBase.h"
#include <folly/MPMCQueue.h> #include <folly/MPMCQueue.h>
#include <grpcpp/grpcpp.h> #include <grpcpp/grpcpp.h>
#include <condition_variable> #include <condition_variable>
#include <iostream>
#include <memory> #include <memory>
#include <string> #include <string>
namespace comm { namespace comm {
namespace network { namespace network {
namespace reactor { namespace reactor {
class BlobPutClientReactor class BlobPutClientReactor
Show All 22 Linespublic:
void scheduleSendingDataChunk(std::unique_ptr<std::string> dataChunk); void scheduleSendingDataChunk(std::unique_ptr<std::string> dataChunk);
std::unique_ptr<grpc::Status> prepareRequest( std::unique_ptr<grpc::Status> prepareRequest(
blob::PutRequest &request, blob::PutRequest &request,
std::shared_ptr<blob::PutResponse> previousResponse) override; std::shared_ptr<blob::PutResponse> previousResponse) override;
void doneCallback() override; void doneCallback() override;
grpc::Status getStatus() const; grpc::Status getStatus() const;
}; };
BlobPutClientReactor::BlobPutClientReactor(
const std::string &holder,
const std::string &hash,
std::condition_variable *terminationNotifier)
: holder(holder),
hash(hash),
dataChunks(folly::MPMCQueue<std::string>(100)),
terminationNotifier(terminationNotifier) {
}
void BlobPutClientReactor::scheduleSendingDataChunk(
std::unique_ptr<std::string> dataChunk) {
if (!this->dataChunks.write(std::move(*dataChunk))) {
throw std::runtime_error(
"Error scheduling sending a data chunk to send to the blob service");
}
}
std::unique_ptr<grpc::Status> BlobPutClientReactor::prepareRequest(
blob::PutRequest &request,
std::shared_ptr<blob::PutResponse> previousResponse) {
if (this->state == State::SEND_HOLDER) {
this->request.set_holder(this->holder);
this->state = State::SEND_HASH;
return nullptr;
}
if (this->state == State::SEND_HASH) {
request.set_blobhash(this->hash);
this->state = State::SEND_CHUNKS;
return nullptr;
}
if (previousResponse->dataexists()) {
return std::make_unique<grpc::Status>(grpc::Status::OK);
}
std::string dataChunk;
this->dataChunks.blockingRead(dataChunk);
if (dataChunk.empty()) {
return std::make_unique<grpc::Status>(grpc::Status::OK);
}
request.set_datachunk(dataChunk);
return nullptr;
}
void BlobPutClientReactor::doneCallback() {
this->terminationNotifier->notify_one();
}
grpc::Status BlobPutClientReactor::getStatus() const {
return this->status;
}
} // namespace reactor } // namespace reactor
} // namespace network } // namespace network
} // namespace comm } // namespace comm
View Options

web/chat/message-preview.react.js

  • This file was copied to services/backup/src/Reactors/client/blob/BlobPutClientReactor.cpp.
#pragma once #pragma once
#include "ClientBidiReactorBase.h"
#include "Constants.h" #include "Constants.h"
#include "../_generated/blob.grpc.pb.h" #include "../_generated/blob.grpc.pb.h"
#include "../_generated/blob.pb.h" #include "../_generated/blob.pb.h"
#include "ClientBidiReactorBase.h"
#include <folly/MPMCQueue.h> #include <folly/MPMCQueue.h>
#include <grpcpp/grpcpp.h> #include <grpcpp/grpcpp.h>
#include <condition_variable> #include <condition_variable>
#include <iostream>
#include <memory> #include <memory>
#include <string> #include <string>
namespace comm { namespace comm {
namespace network { namespace network {
namespace reactor { namespace reactor {
class BlobPutClientReactor class BlobPutClientReactor
Show All 22 Linespublic:
void scheduleSendingDataChunk(std::unique_ptr<std::string> dataChunk); void scheduleSendingDataChunk(std::unique_ptr<std::string> dataChunk);
std::unique_ptr<grpc::Status> prepareRequest( std::unique_ptr<grpc::Status> prepareRequest(
blob::PutRequest &request, blob::PutRequest &request,
std::shared_ptr<blob::PutResponse> previousResponse) override; std::shared_ptr<blob::PutResponse> previousResponse) override;
void doneCallback() override; void doneCallback() override;
grpc::Status getStatus() const; grpc::Status getStatus() const;
}; };
BlobPutClientReactor::BlobPutClientReactor(
const std::string &holder,
const std::string &hash,
std::condition_variable *terminationNotifier)
: holder(holder),
hash(hash),
dataChunks(folly::MPMCQueue<std::string>(100)),
terminationNotifier(terminationNotifier) {
}
void BlobPutClientReactor::scheduleSendingDataChunk(
std::unique_ptr<std::string> dataChunk) {
if (!this->dataChunks.write(std::move(*dataChunk))) {
throw std::runtime_error(
"Error scheduling sending a data chunk to send to the blob service");
}
}
std::unique_ptr<grpc::Status> BlobPutClientReactor::prepareRequest(
blob::PutRequest &request,
std::shared_ptr<blob::PutResponse> previousResponse) {
if (this->state == State::SEND_HOLDER) {
this->request.set_holder(this->holder);
this->state = State::SEND_HASH;
return nullptr;
}
if (this->state == State::SEND_HASH) {
request.set_blobhash(this->hash);
this->state = State::SEND_CHUNKS;
return nullptr;
}
if (previousResponse->dataexists()) {
return std::make_unique<grpc::Status>(grpc::Status::OK);
}
std::string dataChunk;
this->dataChunks.blockingRead(dataChunk);
if (dataChunk.empty()) {
return std::make_unique<grpc::Status>(grpc::Status::OK);
}
request.set_datachunk(dataChunk);
return nullptr;
}
void BlobPutClientReactor::doneCallback() {
this->terminationNotifier->notify_one();
}
grpc::Status BlobPutClientReactor::getStatus() const {
return this->status;
}
} // namespace reactor } // namespace reactor
} // namespace network } // namespace network
} // namespace comm } // namespace comm