Page MenuHomePhabricator
Differential D9368

[lib][web][native] Refactor actions in link-actions.js
ClosedPublic
Actions

Authored by inka on Oct 5 2023, 5:19 AM.
Tags
None
Referenced Files
F6003014: D9368.id.diff
Sat, Apr 19, 12:49 PM
F5979738: D9368.id32647.diff
Sat, Apr 19, 1:48 AM
Unknown Object (File)
Fri, Apr 18, 1:05 PM
Unknown Object (File)
Fri, Apr 18, 12:03 PM
Unknown Object (File)
Fri, Apr 18, 1:13 AM
Unknown Object (File)
Thu, Apr 17, 11:36 PM
Unknown Object (File)
Thu, Apr 17, 12:07 AM
Unknown Object (File)
Sat, Apr 12, 1:38 AM
View All Files
Subscribers
ashoat
tomek
will

Details

Reviewers
michal
kamil
ginsu
Commits
rCOMMbb6079c8aa9e: [lib][web][native] Refactor actions in link-actions.js
Summary

issue: https://linear.app/comm/issue/ENG-4681/create-action-objects-for-all-actions

verifyInviteLink cannot be refactored yet. It will be done in https://linear.app/comm/issue/ENG-5164/refactor-verifyinvitelink-action

Test Plan

Tested that the actions work as expected

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

inka created this revision.Oct 5 2023, 5:19 AM
Herald added subscribers: will, tomek, ashoat. · View Herald TranscriptOct 5 2023, 5:19 AM
Harbormaster returned this revision to the author for changes because remote builds failed.Oct 5 2023, 5:25 AM
Harbormaster failed remote builds in B23001: Diff 31681!
inka added a parent revision: D9356: [lib][web][native] Refactor actions in entry-actions.js.Oct 5 2023, 6:19 AM
inka added a child revision: D9369: [lib][web][native] Refactor actions in message-report-actions.js.
inka updated this revision to Diff 31684.Oct 5 2023, 6:31 AM

Retry

Harbormaster completed remote builds in B23004: Diff 31684.Oct 5 2023, 6:48 AM
inka requested review of this revision.Oct 5 2023, 6:48 AM
michal accepted this revision.Oct 11 2023, 1:19 AM
michal added inline comments.
lib/actions/link-actions.js
95–97 ↗(On Diff #31684)

We shouldn't hide await inside of an another expression

This revision is now accepted and ready to land.Oct 11 2023, 1:19 AM
inka updated this revision to Diff 32620.Nov 2 2023, 7:24 AM

Address review

Harbormaster completed remote builds in B23666: Diff 32620.Nov 2 2023, 7:42 AM
inka updated this revision to Diff 32647.Nov 2 2023, 12:47 PM

Rebase

Harbormaster completed remote builds in B23693: Diff 32647.Nov 2 2023, 1:38 PM
Closed by commit rCOMMbb6079c8aa9e: [lib][web][native] Refactor actions in link-actions.js (authored by inka). · Explain WhyNov 6 2023, 5:44 AM
This revision was automatically updated to reflect the committed changes.
inka added a commit: rCOMMbb6079c8aa9e: [lib][web][native] Refactor actions in link-actions.js.

Revision Contents
Changeset List

PathSize
lib/
actions/
84 lines
hooks/
13 lines
native/
navigation/
9 lines
web/
invite-links/
9 lines

Diff 32759

View Options

lib/actions/link-actions.js

Show All 15 Lines
import { setClientDBStoreActionType } from '../actions/client-db-store-actions.js'; import { setClientDBStoreActionType } from '../actions/client-db-store-actions.js';
import { import {
createEntryActionTypes, createEntryActionTypes,
saveEntryActionTypes, saveEntryActionTypes,
deleteEntryActionTypes, deleteEntryActionTypes,
restoreEntryActionTypes, restoreEntryActionTypes,
} from '../actions/entry-actions.js'; } from '../actions/entry-actions.js';
import { import {
toggleMessagePinActionTypes,
fetchMessagesBeforeCursorActionTypes, fetchMessagesBeforeCursorActionTypes,
fetchMostRecentMessagesActionTypes, fetchMostRecentMessagesActionTypes,
sendTextMessageActionTypes, sendTextMessageActionTypes,
sendMultimediaMessageActionTypes, sendMultimediaMessageActionTypes,
sendReactionMessageActionTypes, sendReactionMessageActionTypes,
sendEditMessageActionTypes, sendEditMessageActionTypes,
saveMessagesActionType, saveMessagesActionType,
processMessagesActionType, processMessagesActionType,
messageStorePruneActionType, messageStorePruneActionType,
createLocalMessageActionType, createLocalMessageActionType,
fetchSingleMostRecentMessagesFromThreadsActionTypes, fetchSingleMostRecentMessagesFromThreadsActionTypes,
} from '../actions/message-actions.js'; } from '../actions/message-actions.js';
import { sendMessageReportActionTypes } from '../actions/message-report-actions.js'; import { sendMessageReportActionTypes } from '../actions/message-report-actions.js';
import { siweAuthActionTypes } from '../actions/siwe-actions.js'; import { siweAuthActionTypes } from '../actions/siwe-actions.js';
import { import {
changeThreadSettingsActionTypes, changeThreadSettingsActionTypes,
deleteThreadActionTypes, deleteThreadActionTypes,
leaveThreadActionTypes, leaveThreadActionTypes,
newThreadActionTypes, newThreadActionTypes,
removeUsersFromThreadActionTypes, removeUsersFromThreadActionTypes,
changeThreadMemberRolesActionTypes, changeThreadMemberRolesActionTypes,
joinThreadActionTypes, joinThreadActionTypes,
toggleMessagePinActionTypes,
} from '../actions/thread-actions.js'; } from '../actions/thread-actions.js';
import { updateMultimediaMessageMediaActionType } from '../actions/upload-actions.js'; import { updateMultimediaMessageMediaActionType } from '../actions/upload-actions.js';
import { import {
logOutActionTypes, logOutActionTypes,
deleteAccountActionTypes, deleteAccountActionTypes,
logInActionTypes, logInActionTypes,
registerActionTypes, registerActionTypes,
} from '../actions/user-actions.js'; } from '../actions/user-actions.js';
▲ Show 20 Lines • Show All 1,707 Lines • Show Last 20 Lines
View Options

lib/hooks/invite-links.js

Show All 15 Lines
import { setClientDBStoreActionType } from '../actions/client-db-store-actions.js'; import { setClientDBStoreActionType } from '../actions/client-db-store-actions.js';
import { import {
createEntryActionTypes, createEntryActionTypes,
saveEntryActionTypes, saveEntryActionTypes,
deleteEntryActionTypes, deleteEntryActionTypes,
restoreEntryActionTypes, restoreEntryActionTypes,
} from '../actions/entry-actions.js'; } from '../actions/entry-actions.js';
import { import {
toggleMessagePinActionTypes,
fetchMessagesBeforeCursorActionTypes, fetchMessagesBeforeCursorActionTypes,
fetchMostRecentMessagesActionTypes, fetchMostRecentMessagesActionTypes,
sendTextMessageActionTypes, sendTextMessageActionTypes,
sendMultimediaMessageActionTypes, sendMultimediaMessageActionTypes,
sendReactionMessageActionTypes, sendReactionMessageActionTypes,
sendEditMessageActionTypes, sendEditMessageActionTypes,
saveMessagesActionType, saveMessagesActionType,
processMessagesActionType, processMessagesActionType,
messageStorePruneActionType, messageStorePruneActionType,
createLocalMessageActionType, createLocalMessageActionType,
fetchSingleMostRecentMessagesFromThreadsActionTypes, fetchSingleMostRecentMessagesFromThreadsActionTypes,
} from '../actions/message-actions.js'; } from '../actions/message-actions.js';
import { sendMessageReportActionTypes } from '../actions/message-report-actions.js'; import { sendMessageReportActionTypes } from '../actions/message-report-actions.js';
import { siweAuthActionTypes } from '../actions/siwe-actions.js'; import { siweAuthActionTypes } from '../actions/siwe-actions.js';
import { import {
changeThreadSettingsActionTypes, changeThreadSettingsActionTypes,
deleteThreadActionTypes, deleteThreadActionTypes,
leaveThreadActionTypes, leaveThreadActionTypes,
newThreadActionTypes, newThreadActionTypes,
removeUsersFromThreadActionTypes, removeUsersFromThreadActionTypes,
changeThreadMemberRolesActionTypes, changeThreadMemberRolesActionTypes,
joinThreadActionTypes, joinThreadActionTypes,
toggleMessagePinActionTypes,
} from '../actions/thread-actions.js'; } from '../actions/thread-actions.js';
import { updateMultimediaMessageMediaActionType } from '../actions/upload-actions.js'; import { updateMultimediaMessageMediaActionType } from '../actions/upload-actions.js';
import { import {
logOutActionTypes, logOutActionTypes,
deleteAccountActionTypes, deleteAccountActionTypes,
logInActionTypes, logInActionTypes,
registerActionTypes, registerActionTypes,
} from '../actions/user-actions.js'; } from '../actions/user-actions.js';
▲ Show 20 Lines • Show All 1,707 Lines • Show Last 20 Lines
View Options

native/navigation/community-drawer-content.react.js

Show All 15 Lines
import { setClientDBStoreActionType } from '../actions/client-db-store-actions.js'; import { setClientDBStoreActionType } from '../actions/client-db-store-actions.js';
import { import {
createEntryActionTypes, createEntryActionTypes,
saveEntryActionTypes, saveEntryActionTypes,
deleteEntryActionTypes, deleteEntryActionTypes,
restoreEntryActionTypes, restoreEntryActionTypes,
} from '../actions/entry-actions.js'; } from '../actions/entry-actions.js';
import { import {
toggleMessagePinActionTypes,
fetchMessagesBeforeCursorActionTypes, fetchMessagesBeforeCursorActionTypes,
fetchMostRecentMessagesActionTypes, fetchMostRecentMessagesActionTypes,
sendTextMessageActionTypes, sendTextMessageActionTypes,
sendMultimediaMessageActionTypes, sendMultimediaMessageActionTypes,
sendReactionMessageActionTypes, sendReactionMessageActionTypes,
sendEditMessageActionTypes, sendEditMessageActionTypes,
saveMessagesActionType, saveMessagesActionType,
processMessagesActionType, processMessagesActionType,
messageStorePruneActionType, messageStorePruneActionType,
createLocalMessageActionType, createLocalMessageActionType,
fetchSingleMostRecentMessagesFromThreadsActionTypes, fetchSingleMostRecentMessagesFromThreadsActionTypes,
} from '../actions/message-actions.js'; } from '../actions/message-actions.js';
import { sendMessageReportActionTypes } from '../actions/message-report-actions.js'; import { sendMessageReportActionTypes } from '../actions/message-report-actions.js';
import { siweAuthActionTypes } from '../actions/siwe-actions.js'; import { siweAuthActionTypes } from '../actions/siwe-actions.js';
import { import {
changeThreadSettingsActionTypes, changeThreadSettingsActionTypes,
deleteThreadActionTypes, deleteThreadActionTypes,
leaveThreadActionTypes, leaveThreadActionTypes,
newThreadActionTypes, newThreadActionTypes,
removeUsersFromThreadActionTypes, removeUsersFromThreadActionTypes,
changeThreadMemberRolesActionTypes, changeThreadMemberRolesActionTypes,
joinThreadActionTypes, joinThreadActionTypes,
toggleMessagePinActionTypes,
} from '../actions/thread-actions.js'; } from '../actions/thread-actions.js';
import { updateMultimediaMessageMediaActionType } from '../actions/upload-actions.js'; import { updateMultimediaMessageMediaActionType } from '../actions/upload-actions.js';
import { import {
logOutActionTypes, logOutActionTypes,
deleteAccountActionTypes, deleteAccountActionTypes,
logInActionTypes, logInActionTypes,
registerActionTypes, registerActionTypes,
} from '../actions/user-actions.js'; } from '../actions/user-actions.js';
▲ Show 20 Lines • Show All 1,707 Lines • Show Last 20 Lines
View Options

web/invite-links/invite-links-refresher.react.js

Show All 15 Lines
import { setClientDBStoreActionType } from '../actions/client-db-store-actions.js'; import { setClientDBStoreActionType } from '../actions/client-db-store-actions.js';
import { import {
createEntryActionTypes, createEntryActionTypes,
saveEntryActionTypes, saveEntryActionTypes,
deleteEntryActionTypes, deleteEntryActionTypes,
restoreEntryActionTypes, restoreEntryActionTypes,
} from '../actions/entry-actions.js'; } from '../actions/entry-actions.js';
import { import {
toggleMessagePinActionTypes,
fetchMessagesBeforeCursorActionTypes, fetchMessagesBeforeCursorActionTypes,
fetchMostRecentMessagesActionTypes, fetchMostRecentMessagesActionTypes,
sendTextMessageActionTypes, sendTextMessageActionTypes,
sendMultimediaMessageActionTypes, sendMultimediaMessageActionTypes,
sendReactionMessageActionTypes, sendReactionMessageActionTypes,
sendEditMessageActionTypes, sendEditMessageActionTypes,
saveMessagesActionType, saveMessagesActionType,
processMessagesActionType, processMessagesActionType,
messageStorePruneActionType, messageStorePruneActionType,
createLocalMessageActionType, createLocalMessageActionType,
fetchSingleMostRecentMessagesFromThreadsActionTypes, fetchSingleMostRecentMessagesFromThreadsActionTypes,
} from '../actions/message-actions.js'; } from '../actions/message-actions.js';
import { sendMessageReportActionTypes } from '../actions/message-report-actions.js'; import { sendMessageReportActionTypes } from '../actions/message-report-actions.js';
import { siweAuthActionTypes } from '../actions/siwe-actions.js'; import { siweAuthActionTypes } from '../actions/siwe-actions.js';
import { import {
changeThreadSettingsActionTypes, changeThreadSettingsActionTypes,
deleteThreadActionTypes, deleteThreadActionTypes,
leaveThreadActionTypes, leaveThreadActionTypes,
newThreadActionTypes, newThreadActionTypes,
removeUsersFromThreadActionTypes, removeUsersFromThreadActionTypes,
changeThreadMemberRolesActionTypes, changeThreadMemberRolesActionTypes,
joinThreadActionTypes, joinThreadActionTypes,
toggleMessagePinActionTypes,
} from '../actions/thread-actions.js'; } from '../actions/thread-actions.js';
import { updateMultimediaMessageMediaActionType } from '../actions/upload-actions.js'; import { updateMultimediaMessageMediaActionType } from '../actions/upload-actions.js';
import { import {
logOutActionTypes, logOutActionTypes,
deleteAccountActionTypes, deleteAccountActionTypes,
logInActionTypes, logInActionTypes,
registerActionTypes, registerActionTypes,
} from '../actions/user-actions.js'; } from '../actions/user-actions.js';
▲ Show 20 Lines • Show All 1,707 Lines • Show Last 20 Lines