[web] Update `LoginForm` `useEffect` early-exit logic to include `notificationIdentityPublicKeys`
ClosedPublic
Actions

Authored by atul on Feb 23 2023, 7:20 PM.

Details

Reviewers

ashoat
tomek
marcin

Commits

rCOMM5e9a5a925a98: [web] Update `LoginForm` `useEffect` early-exit logic to include…

Summary

Instead of just considering the primaryIdentityPublicKey, we check wither the primaryIdentityPublicKeys and notificationIdentityPublicKeys objects are set (eg not null or undefined).

Depends on D6878

Test Plan

Logged values, things were as expected.

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

atul created this revision.Feb 23 2023, 7:20 PM

atul published this revision for review.Feb 23 2023, 7:20 PM

atul edited the summary of this revision. (Show Details)

atul added a parent revision: D6878: [web] Dispatch `setNotificationIdentityKeys` in `LoginForm`.

atul added a child revision: D6880: [web] Update `primaryIdentityPublicKey[s]` selector to capture `primaryIdentityPublicKeys`.

Harbormaster completed remote builds in B16864: Diff 23043.Feb 23 2023, 7:35 PM

atul added a reviewer: marcin.Feb 24 2023, 2:21 AM

ashoat accepted this revision.Feb 24 2023, 8:15 AM

This revision is now accepted and ready to land.Feb 24 2023, 8:15 AM

Closed by commit rCOMM5e9a5a925a98: [web] Update `LoginForm` `useEffect` early-exit logic to include… (authored by atul). · Explain WhyFeb 24 2023, 2:39 PM

This revision was automatically updated to reflect the committed changes.

atul added a commit: rCOMM5e9a5a925a98: [web] Update `LoginForm` `useEffect` early-exit logic to include….

Revision Contents
Changeset List

Path

Size

web/

account/

log-in-form.react.js

15 lines

Diff 23087

View Options

web/account/log-in-form.react.js

	#include "MessageOperationsUtilities.h"			#include "MessageOperationsUtilities.h"
	#include "MessageSpecs.h"			#include "MessageSpecs.h"

	#include <folly/String.h>			#include <folly/String.h>

	namespace comm {			namespace comm {
				ClientDBMessageInfo
				MessageOperationsUtilities::translateRawMessageInfoToClientDBMessageInfo(
				const folly::dynamic &rawMessageInfo) {
				std::string id = rawMessageInfo.count("id")
				? rawMessageInfo["id"].asString()
				: rawMessageInfo["localID"].asString();
				std::string thread = rawMessageInfo["threadID"].asString();
				std::string user = rawMessageInfo["creatorID"].asString();

				std::unique_ptr<std::string> localID = nullptr;
				if (rawMessageInfo.count("localID")) {
				localID =
				std::make_unique<std::string>(rawMessageInfo["localID"].asString());
				}

				int type = rawMessageInfo["type"].asInt();
				MessageType messageType = static_cast<MessageType>(type);
				int64_t time = rawMessageInfo["time"].asInt();

				std::unique_ptr<int> futureType = nullptr;
				if (messageType == MessageType::UNSUPPORTED) {
				futureType = std::make_unique<int>(
				rawMessageInfo["unsupportedMessageInfo"]["type"].asInt());
				}

				std::unique_ptr<std::string> content = nullptr;
				if (messageSpecsHolder.find(messageType) != messageSpecsHolder.end()) {
				content = messageSpecsHolder.at(messageType)
				->messageContentForClientDB(rawMessageInfo);
				}
				std::vector<Media> mediaVector;
				if (messageType == MessageType::IMAGES \|\|
				messageType == MessageType::MULTIMEDIA) {
				for (const auto &media : rawMessageInfo["media"]) {
				mediaVector.push_back(
				translateMediaToClientDBMediaInfo(media, id, thread));
				}
				}
				return {
				Message{
				id,
				std::move(localID),
				thread,
				user,
				type,
				std::move(futureType),
				std::move(content),
				time},
				std::move(mediaVector)};
				}

	Media MessageOperationsUtilities::translateMediaToClientDBMediaInfo(			Media MessageOperationsUtilities::translateMediaToClientDBMediaInfo(
	const folly::dynamic &rawMediaInfo,			const folly::dynamic &rawMediaInfo,
	const std::string &container,			const std::string &container,
	const std::string &thread) {			const std::string &thread) {
	std::string id = rawMediaInfo["id"].asString();			std::string id = rawMediaInfo["id"].asString();
	std::string uri = rawMediaInfo["uri"].asString();			std::string uri = rawMediaInfo["uri"].asString();
	std::string type = rawMediaInfo["type"].asString();			std::string type = rawMediaInfo["type"].asString();
	folly::dynamic extrasData =			folly::dynamic extrasData =
	Show All 10 Lines

[web] Update `LoginForm` `useEffect` early-exit logic to include `notificationIdentityPublicKeys`ClosedPublicActions