Paths

Table of Contentst

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

Authored by rohan on Sep 28 2023, 7:42 PM.

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, 7:42 PM

Herald added subscribers: tomek, ashoat. · View Herald TranscriptSep 28 2023, 7:42 PM

Harbormaster returned this revision to the author for changes because remote builds failed.Sep 28 2023, 7: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, 7:47 PM

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

Harbormaster completed remote builds in B22904: Diff 31535.Sep 29 2023, 12:51 AM

ginsu accepted this revision.Sep 29 2023, 7:07 PM

This revision is now accepted and ready to land.Sep 29 2023, 7:07 PM

@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, 2:58 PM

rohan removed a child revision: D9324: [lib] Update types for SIDEBAR_SOURCE messages to reflect isInvalidSidebarSource.

Revision Contents
Changeset List

Path

Size

keyserver/

src/

creators/

thread-creator.js

2 lines

fetchers/

message-fetchers.js

6 lines

lib/

shared/

message-utils.js

12 lines

message-utils.test.js

messages/

sidebar-source-message-spec.js

2 lines

thread-utils.js

3 lines

types/

message-types.js

11 lines

	types/
	shared/

	message-types.test.js
	message-utils.test.js

48 lines

Diff 31535

View Options

keyserver/src/creators/thread-creator.js

View Options

keyserver/src/fetchers/message-fetchers.js

View Options

lib/shared/message-utils.js

View Options

lib/shared/message-utils.test.js

View Options

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

View Options

lib/shared/thread-utils.js

View Options

lib/types/message-types.js

View Options

lib/types/message-types.test.js

[keyserver/lib] Move isInvalidSidebarSource from message-utils to message-typesAbandonedPublicActions

Details

Diff Detail

Event Timeline

Revision ContentsChangeset List

Diff 31535

keyserver/src/creators/thread-creator.js

keyserver/src/fetchers/message-fetchers.js

lib/shared/message-utils.js

lib/shared/message-utils.test.js

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

lib/shared/thread-utils.js

lib/types/message-types.js

lib/types/message-types.test.js

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

Revision Contents
Changeset List