Paths

Table of Contentst

[native] rename MessageResultsScreen to PinnedMessagesScreen
ClosedPublic
Actions

Authored by ginsu on Feb 7 2024, 8:52 AM.

Details

Reviewers

atul
inka

Commits

rCOMM9630b275ef41: [native] rename MessageResultsScreen to PinnedMessagesScreen

Summary

In D10345 I got some feedback to update the MessageResultsScreen to PinnedMessagesScreen

Linear task: https://linear.app/comm/issue/ENG-6356/rename-messageresultsscreen-on-native-to-be-pinned-messages-screen

Test Plan

flow + confirmed PinnedMessagesScreen has no regressions

Diff Detail

Repository

rCOMM Comm

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

ginsu created this revision.Feb 7 2024, 8:52 AM

Herald added subscribers: tomek, ashoat. · View Herald TranscriptFeb 7 2024, 8:52 AM

ginsu edited the summary of this revision. (Show Details)Feb 7 2024, 8:53 AM

ginsu added reviewers: atul, inka.

Harbormaster completed remote builds in B26611: Diff 36768.Feb 7 2024, 9:20 AM

ginsu requested review of this revision.Feb 7 2024, 9:20 AM

Skimmed, assuming this is straightforward refactor.

This revision is now accepted and ready to land.Feb 7 2024, 8:58 PM

rebase before landing

This revision was landed with ongoing or failed builds.Feb 7 2024, 10:08 PM

Closed by commit rCOMM9630b275ef41: [native] rename MessageResultsScreen to PinnedMessagesScreen. · Explain Why

This revision was automatically updated to reflect the committed changes.

ginsu added a commit: rCOMM9630b275ef41: [native] rename MessageResultsScreen to PinnedMessagesScreen.

Harbormaster completed remote builds in B26636: Diff 36808.Feb 7 2024, 10:18 PM

Revision Contents
Changeset List

Path

Size

native/

chat/

chat.react.js

12 lines

message-header.react.js

2 lines

message-list-container.react.js

8 lines

message-result.react.js

4 lines

message-results-screen.react.js

message.react.js

4 lines

	pinned-messages-screen.react.js
	message-results-screen.react.js

12 lines

robotext-message.react.js

4 lines

text-message.react.js

4 lines

navigation/

nav-selectors.js

4 lines

route-names.js

8 lines

Diff 36768

View Options

native/chat/chat.react.js

View Options

native/chat/message-header.react.js

View Options

native/chat/message-list-container.react.js

View Options

native/chat/message-result.react.js

View Options

native/chat/message-results-screen.react.js

View Options

native/chat/message.react.js

View Options

native/chat/pinned-messages-screen.react.js

View Options

native/chat/robotext-message.react.js

View Options

native/chat/text-message.react.js

View Options

native/navigation/nav-selectors.js

View Options

native/navigation/route-names.js

[native] rename MessageResultsScreen to PinnedMessagesScreenClosedPublicActions

Details

Diff Detail

Event Timeline

Revision ContentsChangeset List

Diff 36768

native/chat/chat.react.js

native/chat/message-header.react.js

native/chat/message-list-container.react.js

native/chat/message-result.react.js

native/chat/message-results-screen.react.js

native/chat/message.react.js

native/chat/pinned-messages-screen.react.js

native/chat/robotext-message.react.js

native/chat/text-message.react.js

native/navigation/nav-selectors.js

native/navigation/route-names.js

[native] rename MessageResultsScreen to PinnedMessagesScreen
ClosedPublic
Actions

Revision Contents
Changeset List