Page MenuHomePhabricator
Differential D11208

[lib] Deprecate `useRoleDeletableAndEditableStatus` and determine `is[Editable/Deletable]` directly in `RoleActionsMenu`
ClosedPublic
Actions

Authored by atul on Mar 1 2024, 12:57 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Mar 10, 12:41 PM
Unknown Object (File)
Feb 22 2025, 2:00 AM
Unknown Object (File)
Feb 22 2025, 2:00 AM
Unknown Object (File)
Feb 22 2025, 2:00 AM
Unknown Object (File)
Feb 22 2025, 2:00 AM
Unknown Object (File)
Feb 22 2025, 2:00 AM
Unknown Object (File)
Feb 7 2025, 12:20 PM
Unknown Object (File)
Feb 7 2025, 4:32 AM
View All 88 Files
Subscribers
ashoat

Details

Reviewers
ginsu
tomek
kamil
michal
Commits
rCOMM49bcd23b617f: [lib] Deprecate `useRoleDeletableAndEditableStatus` and determine `is…
Summary

Context: https://linear.app/comm/issue/ENG-6949/update-roledeletableandeditablestatus-to-use-specialrole-field

We can use specialRole field and roleIsDefaultRole and roleIsAdminRole utilities directly instead of doing this in a separate hook.

Depends on D11206

Test Plan

Make sure that RoleActionsMenu appears and behaves as expected.

Will put screenshots in here shortly:

Diff Detail

Repository
rCOMM Comm
Branch
arcpatch-D11208 (branched from master)
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

atul created this revision.Mar 1 2024, 12:57 PM
Herald added a subscriber: ashoat. · View Herald TranscriptMar 1 2024, 12:58 PM
Harbormaster completed remote builds in B27257: Diff 37734.Mar 1 2024, 1:13 PM
atul requested review of this revision.Mar 1 2024, 1:13 PM
atul added a child revision: D11209: [lib] Update `threadHasAdminRole` to consider `specialField`.Mar 1 2024, 1:16 PM
tomek accepted this revision.Mar 4 2024, 1:05 AM

We should reintroduce the util (probably as a simple function) if we would need to decide if we can edit / delete a role in some other places.

This revision is now accepted and ready to land.Mar 4 2024, 1:05 AM
atul added a comment.Mar 19 2024, 10:32 AM

This is safe to land because we will fall back to previous roleIsAdminRole and roleIsDefaultRole logic if specialRole field is missing (as it will on web when landed prior to "flipping the switch").

In D11208#324212, @tomek wrote:

We should reintroduce the util (probably as a simple function) if we would need to decide if we can edit / delete a role in some other places.

Yeah, agree but figure this reduces indirection for time being when there's a single instance.

atul updated this revision to Diff 38199.Mar 19 2024, 10:35 AM

rebase around and land

Harbormaster completed remote builds in B27605: Diff 38199.Mar 19 2024, 10:49 AM
Closed by commit rCOMM49bcd23b617f: [lib] Deprecate `useRoleDeletableAndEditableStatus` and determine `is… (authored by atul). · Explain WhyMar 19 2024, 10:51 AM
This revision was automatically updated to reflect the committed changes.
atul added a commit: rCOMM49bcd23b617f: [lib] Deprecate `useRoleDeletableAndEditableStatus` and determine `is….

Revision Contents
Changeset List

PathSize
lib/
utils/
21 lines
web/
roles/
15 lines

Diff 38199

View Options

lib/utils/role-utils.js

// @flow // @flow
import _keyBy from 'lodash/fp/keyBy.js';
import type { ClientDBMessageStoreOperation } from 'lib/ops/message-store-ops.js'; import type { ClientDBMessageStoreOperation } from 'lib/ops/message-store-ops.js';
import type { ClientDBThreadStoreOperation } from 'lib/ops/thread-store-ops.js';
import type { import type {
RawMessageInfo, RawMessageInfo,
ClientDBMessageInfo, ClientDBMessageInfo,
ClientDBThreadMessageInfo, ClientDBThreadMessageInfo,
} from 'lib/types/message-types.js'; } from 'lib/types/message-types.js';
import type {
ClientDBThreadInfo,
RawThreadInfos,
} from 'lib/types/thread-types.js';
import { import {
translateClientDBMessageInfoToRawMessageInfo, translateClientDBMessageInfoToRawMessageInfo,
translateRawMessageInfoToClientDBMessageInfo, translateRawMessageInfoToClientDBMessageInfo,
translateClientDBThreadMessageInfos, translateClientDBThreadMessageInfos,
translateThreadMessageInfoToClientDBThreadMessageInfo, translateThreadMessageInfoToClientDBThreadMessageInfo,
type TranslatedThreadMessageInfos, type TranslatedThreadMessageInfos,
} from 'lib/utils/message-ops-utils.js'; } from 'lib/utils/message-ops-utils.js';
import { entries } from 'lib/utils/objects.js'; import { entries, values } from 'lib/utils/objects.js';
import {
convertClientDBThreadInfoToRawThreadInfo,
convertRawThreadInfoToClientDBThreadInfo,
} from 'lib/utils/thread-ops-utils.js';
function createUpdateDBOpsForThreadStoreThreadInfos(
clientDBThreadInfos: $ReadOnlyArray<ClientDBThreadInfo>,
migrationFunc: RawThreadInfos => RawThreadInfos,
): $ReadOnlyArray<ClientDBThreadStoreOperation> {
// 1. Translate `ClientDBThreadInfo`s to `RawThreadInfo`s.
const rawThreadInfos = clientDBThreadInfos.map(
convertClientDBThreadInfoToRawThreadInfo,
);
// 2. Convert `RawThreadInfo`s to a map of `threadID` => `threadInfo`.
const keyedRawThreadInfos = _keyBy('id')(rawThreadInfos);
// 3. Apply `migrationFunc` to `ThreadInfo`s.
const updatedKeyedRawThreadInfos = migrationFunc(keyedRawThreadInfos);
// 4. Convert the updated `RawThreadInfos` back into an array.
const updatedKeyedRawThreadInfosArray = values(updatedKeyedRawThreadInfos);
// 5. Translate `RawThreadInfo`s back to `ClientDBThreadInfo`s.
const updatedClientDBThreadInfos = updatedKeyedRawThreadInfosArray.map(
convertRawThreadInfoToClientDBThreadInfo,
);
// 6. Construct `replace` `ClientDBThreadStoreOperation`s.
const replaceThreadOperations = updatedClientDBThreadInfos.map(thread => ({
type: 'replace',
payload: thread,
}));
// 7. Prepend `replaceThreadOperations` with `remove_all` op and return.
return [{ type: 'remove_all' }, ...replaceThreadOperations];
}
function createUpdateDBOpsForMessageStoreMessages( function createUpdateDBOpsForMessageStoreMessages(
clientDBMessageInfos: $ReadOnlyArray<ClientDBMessageInfo>, clientDBMessageInfos: $ReadOnlyArray<ClientDBMessageInfo>,
migrationFunc: ( migrationFunc: (
$ReadOnlyArray<RawMessageInfo>, $ReadOnlyArray<RawMessageInfo>,
) => $ReadOnlyArray<RawMessageInfo>, ) => $ReadOnlyArray<RawMessageInfo>,
): $ReadOnlyArray<ClientDBMessageStoreOperation> { ): $ReadOnlyArray<ClientDBMessageStoreOperation> {
const rawMessageInfos = clientDBMessageInfos.map( const rawMessageInfos = clientDBMessageInfos.map(
Show All 39 Lines return [
translateThreadMessageInfoToClientDBThreadMessageInfo(id, thread), translateThreadMessageInfoToClientDBThreadMessageInfo(id, thread),
), ),
}, },
}, },
]; ];
} }
export { export {
createUpdateDBOpsForThreadStoreThreadInfos,
createUpdateDBOpsForMessageStoreMessages, createUpdateDBOpsForMessageStoreMessages,
createUpdateDBOpsForMessageStoreThreads, createUpdateDBOpsForMessageStoreThreads,
}; };
View Options

web/roles/role-actions-menu.react.js

// @flow // @flow
import _keyBy from 'lodash/fp/keyBy.js';
import type { ClientDBMessageStoreOperation } from 'lib/ops/message-store-ops.js'; import type { ClientDBMessageStoreOperation } from 'lib/ops/message-store-ops.js';
import type { ClientDBThreadStoreOperation } from 'lib/ops/thread-store-ops.js';
import type { import type {
RawMessageInfo, RawMessageInfo,
ClientDBMessageInfo, ClientDBMessageInfo,
ClientDBThreadMessageInfo, ClientDBThreadMessageInfo,
} from 'lib/types/message-types.js'; } from 'lib/types/message-types.js';
import type {
ClientDBThreadInfo,
RawThreadInfos,
} from 'lib/types/thread-types.js';
import { import {
translateClientDBMessageInfoToRawMessageInfo, translateClientDBMessageInfoToRawMessageInfo,
translateRawMessageInfoToClientDBMessageInfo, translateRawMessageInfoToClientDBMessageInfo,
translateClientDBThreadMessageInfos, translateClientDBThreadMessageInfos,
translateThreadMessageInfoToClientDBThreadMessageInfo, translateThreadMessageInfoToClientDBThreadMessageInfo,
type TranslatedThreadMessageInfos, type TranslatedThreadMessageInfos,
} from 'lib/utils/message-ops-utils.js'; } from 'lib/utils/message-ops-utils.js';
import { entries } from 'lib/utils/objects.js'; import { entries, values } from 'lib/utils/objects.js';
import {
convertClientDBThreadInfoToRawThreadInfo,
convertRawThreadInfoToClientDBThreadInfo,
} from 'lib/utils/thread-ops-utils.js';
function createUpdateDBOpsForThreadStoreThreadInfos(
clientDBThreadInfos: $ReadOnlyArray<ClientDBThreadInfo>,
migrationFunc: RawThreadInfos => RawThreadInfos,
): $ReadOnlyArray<ClientDBThreadStoreOperation> {
// 1. Translate `ClientDBThreadInfo`s to `RawThreadInfo`s.
const rawThreadInfos = clientDBThreadInfos.map(
convertClientDBThreadInfoToRawThreadInfo,
);
// 2. Convert `RawThreadInfo`s to a map of `threadID` => `threadInfo`.
const keyedRawThreadInfos = _keyBy('id')(rawThreadInfos);
// 3. Apply `migrationFunc` to `ThreadInfo`s.
const updatedKeyedRawThreadInfos = migrationFunc(keyedRawThreadInfos);
// 4. Convert the updated `RawThreadInfos` back into an array.
const updatedKeyedRawThreadInfosArray = values(updatedKeyedRawThreadInfos);
// 5. Translate `RawThreadInfo`s back to `ClientDBThreadInfo`s.
const updatedClientDBThreadInfos = updatedKeyedRawThreadInfosArray.map(
convertRawThreadInfoToClientDBThreadInfo,
);
// 6. Construct `replace` `ClientDBThreadStoreOperation`s.
const replaceThreadOperations = updatedClientDBThreadInfos.map(thread => ({
type: 'replace',
payload: thread,
}));
// 7. Prepend `replaceThreadOperations` with `remove_all` op and return.
return [{ type: 'remove_all' }, ...replaceThreadOperations];
}
function createUpdateDBOpsForMessageStoreMessages( function createUpdateDBOpsForMessageStoreMessages(
clientDBMessageInfos: $ReadOnlyArray<ClientDBMessageInfo>, clientDBMessageInfos: $ReadOnlyArray<ClientDBMessageInfo>,
migrationFunc: ( migrationFunc: (
$ReadOnlyArray<RawMessageInfo>, $ReadOnlyArray<RawMessageInfo>,
) => $ReadOnlyArray<RawMessageInfo>, ) => $ReadOnlyArray<RawMessageInfo>,
): $ReadOnlyArray<ClientDBMessageStoreOperation> { ): $ReadOnlyArray<ClientDBMessageStoreOperation> {
const rawMessageInfos = clientDBMessageInfos.map( const rawMessageInfos = clientDBMessageInfos.map(
Show All 39 Lines return [
translateThreadMessageInfoToClientDBThreadMessageInfo(id, thread), translateThreadMessageInfoToClientDBThreadMessageInfo(id, thread),
), ),
}, },
}, },
]; ];
} }
export { export {
createUpdateDBOpsForThreadStoreThreadInfos,
createUpdateDBOpsForMessageStoreMessages, createUpdateDBOpsForMessageStoreMessages,
createUpdateDBOpsForMessageStoreThreads, createUpdateDBOpsForMessageStoreThreads,
}; };