Page MenuHomePhabricator
Differential D3768

[web] Move `onChangeThreadType` up from `ThreadSettingsModal` to `ConnectedThreadSettingsModal`
ClosedPublic
Actions

Authored by atul on Apr 18 2022, 3:04 PM.
Tags
None
Referenced Files
F5100442: D3768.id11750.diff
Mon, Mar 31, 1:00 PM
F5100436: D3768.id11751.diff
Mon, Mar 31, 12:58 PM
Unknown Object (File)
Sun, Mar 30, 9:52 AM
Unknown Object (File)
Sat, Mar 29, 7:11 AM
Unknown Object (File)
Sun, Mar 23, 5:32 PM
Unknown Object (File)
Sat, Mar 22, 12:48 AM
Unknown Object (File)
Wed, Mar 19, 4:55 AM
Unknown Object (File)
Thu, Mar 13, 2:28 AM
View All Files
Subscribers
• abosh
• adrian
ashoat
• benschac
• karol

Details

Reviewers
• jacek
tomek
Commits
rCOMM669e261b8c74: [web] Move `onChangeThreadType` up from `ThreadSettingsModal` to…
Summary

Move onChangeThreadType out from inner class component to outer "Connected" functional component.

As part of the work to refactor ThreadSettingsModal... specifically to turn it into a functional component.

Depends on D3767

Test Plan
  1. Open ThreadSettingsModal
  2. Change the thread type
  3. Log values in onChangeThreadtype to ensure that they continue to be as expected
  4. Hit save after changing the color and ensure that the thread description changed as expected

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

atul created this revision.Apr 18 2022, 3:04 PM
Herald added subscribers: • abosh, • benschac, • karol and 2 others. · View Herald TranscriptApr 18 2022, 3:04 PM
Harbormaster completed remote builds in B8281: Diff 11577.Apr 18 2022, 3:09 PM
atul requested review of this revision.Apr 18 2022, 3:09 PM
atul added a child revision: D3769: [web] Move `onChangeAccountPassword` up from `ThreadSettingsModal` to `ConnectedThreadSettingsModal`.Apr 18 2022, 3:25 PM
atul updated this revision to Diff 11649.Apr 19 2022, 3:04 PM

rebase

Harbormaster completed remote builds in B8339: Diff 11649.Apr 19 2022, 3:23 PM
tomek accepted this revision.Apr 21 2022, 3:09 AM
tomek added inline comments.
web/modals/threads/thread-settings-modal.react.js
449–459
This revision is now accepted and ready to land.Apr 21 2022, 3:09 AM
atul updated this revision to Diff 11749.Apr 21 2022, 11:06 AM

rebase before addressing feedback

atul updated this revision to Diff 11750.Apr 21 2022, 11:08 AM

address feedback

Harbormaster completed remote builds in B8421: Diff 11749.Apr 21 2022, 11:11 AM
Harbormaster completed remote builds in B8422: Diff 11750.Apr 21 2022, 11:15 AM
Closed by commit rCOMM669e261b8c74: [web] Move `onChangeThreadType` up from `ThreadSettingsModal` to… (authored by atul). · Explain WhyApr 21 2022, 11:15 AM
This revision was automatically updated to reflect the committed changes.
atul added a commit: rCOMM669e261b8c74: [web] Move `onChangeThreadType` up from `ThreadSettingsModal` to….

Revision Contents
Changeset List

PathSize
web/
modals/
threads/
30 lines

Diff 11649

View Options

web/modals/threads/thread-settings-modal.react.js

#include "CommQueryExecutor.h" #include "CommQueryExecutor.h"
#include "Logger.h"
#include "sqlite_orm.h" #include "sqlite_orm.h"
#include <sqlite3.h> #include <sqlite3.h>
#include <cstdio> #include <cstdio>
#include <iostream> #include <iostream>
#include <string> #include <string>
#ifdef EMSCRIPTEN #ifdef EMSCRIPTEN
#include <emscripten/bind.h> #include <emscripten/bind.h>
#endif #endif
namespace comm { namespace comm {
using namespace sqlite_orm; using namespace sqlite_orm;
std::string DB_PATH = "test.sqlite"; std::string DB_PATH = "test.sqlite";
struct Test { struct Test {
std::string id; std::string id;
std::string data; std::string data;
}; };
std::string CommQueryExecutor::testDBOperation() { std::string CommQueryExecutor::testDBOperation() {
Logger::log("Beginning test db operation!");
sqlite3 *db; sqlite3 *db;
sqlite3_open(DB_PATH.c_str(), &db); sqlite3_open(DB_PATH.c_str(), &db);
std::string createTableQuery = std::string createTableQuery =
"CREATE TABLE IF NOT EXISTS test (" "CREATE TABLE IF NOT EXISTS test ("
" id TEXT UNIQUE PRIMARY KEY NOT NULL," " id TEXT UNIQUE PRIMARY KEY NOT NULL,"
" data TEXT NOT NULL" " data TEXT NOT NULL"
");"; ");";
▲ Show 20 Lines • Show All 44 Lines • Show Last 20 Lines