Page MenuHomePhabricator
Differential D9323

[keyserver/lib] Move isInvalidSidebarSource from message-utils to message-types
AbandonedPublic
Actions

Authored by rohan on Sep 28 2023, 12:42 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Nov 8, 9:25 PM
Unknown Object (File)
Mon, Oct 28, 9:43 AM
Unknown Object (File)
Sat, Oct 19, 4:19 AM
Unknown Object (File)
Sat, Oct 19, 4:18 AM
Unknown Object (File)
Sat, Oct 19, 4:02 AM
Unknown Object (File)
Wed, Oct 16, 8:17 PM
Unknown Object (File)
Oct 12 2024, 8:27 AM
Unknown Object (File)
Sep 15 2024, 1:14 PM
View All 46 Files
Subscribers
ashoat
michal
tomek

Details

Reviewers
atul
ginsu
Summary

Just a simple moving diff since it makes more sense to be with message types in my opinion rather than message utils.

Depends on D9322

Test Plan

Ran flow and manually checked the imports were updated

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

rohan created this revision.Sep 28 2023, 12:42 PM
Herald added subscribers: tomek, ashoat. · View Herald TranscriptSep 28 2023, 12:42 PM
Harbormaster returned this revision to the author for changes because remote builds failed.Sep 28 2023, 12:44 PM
Harbormaster failed remote builds in B22904: Diff 31535!
rohan added a child revision: D9324: [lib] Update types for SIDEBAR_SOURCE messages to reflect isInvalidSidebarSource.Sep 28 2023, 12:47 PM
rohan requested review of this revision.Sep 28 2023, 1:34 PM

I'll rerun CI when it's back up, requesting review in the meantime

Harbormaster completed remote builds in B22904: Diff 31535.Sep 28 2023, 5:51 PM
ginsu accepted this revision.Sep 29 2023, 12:07 PM
This revision is now accepted and ready to land.Sep 29 2023, 12:07 PM
rohan abandoned this revision.Oct 10 2023, 7:57 AM
rohan added a subscriber: michal.

@michal pointed out that doing this will later cause a circular dependency, so I'm keeping it in message-utils

rohan removed a parent revision: D9322: [keyserver/lib] Dedup isMessageSidebarSourceReactionEditOrPin into isInvalidSidebarSource.Oct 10 2023, 7:58 AM
rohan removed a child revision: D9324: [lib] Update types for SIDEBAR_SOURCE messages to reflect isInvalidSidebarSource.

Revision Contents
Changeset List

PathSize
keyserver/
src/
creators/
2 lines
fetchers/
6 lines
lib/
shared/
12 lines
messages/
2 lines
3 lines
types/
11 lines
types/
shared/
message-types.test.js
message-utils.test.js
48 lines

Diff 31535

View Options

keyserver/src/creators/thread-creator.js

Loading...
View Options

keyserver/src/fetchers/message-fetchers.js

Loading...
View Options

lib/shared/message-utils.js

Loading...
View Options

lib/shared/message-utils.test.js

Loading...
View Options

lib/shared/messages/sidebar-source-message-spec.js

Loading...
View Options

lib/shared/thread-utils.js

Loading...
View Options

lib/types/message-types.js

Loading...
View Options

lib/types/message-types.test.js

Loading...