Paths

Table of Contentst

[web] Rename mostRecentReadThread to mostRecentlyReadThread
ClosedPublic
Actions

Authored by tomek on Mar 9 2022, 5:49 PM.

Details

Reviewers

• benschac
atul
• jacek

Commits

rCOMMd44fbb6a81ca: [web] Rename mostRecentReadThread to mostRecentlyReadThread

Summary

In D3341 we realized that it might be a good idea to rename mostRecentReadThread to mostRecentlyReadThread. This diff contains this rename with the corresponding selector rename to keep things consistent.
Depends on D3386

Test Plan

Flow

Diff Detail

Repository

rCOMM Comm

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

tomek created this revision.Mar 9 2022, 5:49 PM

Herald added subscribers: • karol, • adrian, ashoat. · View Herald TranscriptMar 9 2022, 5:49 PM

Harbormaster completed remote builds in B7260: Diff 10231.Mar 9 2022, 5:54 PM

tomek requested review of this revision.Mar 9 2022, 5:54 PM

tomek added a child revision: D3388: [web] Address remaining code review comments from D3341.Mar 9 2022, 5:55 PM

tomek mentioned this in D3341: [web] Make NavigationPanel reusable.Mar 9 2022, 5:55 PM

• benschac accepted this revision.Mar 9 2022, 6:31 PM

This revision is now accepted and ready to land.Mar 9 2022, 6:31 PM

atul accepted this revision.Mar 9 2022, 7:53 PM

Closed by commit rCOMMd44fbb6a81ca: [web] Rename mostRecentReadThread to mostRecentlyReadThread. · Explain WhyMar 11 2022, 11:45 AM

This revision was automatically updated to reflect the committed changes.

tomek added a commit: rCOMMd44fbb6a81ca: [web] Rename mostRecentReadThread to mostRecentlyReadThread.

Revision Contents
Changeset List

Path

Size

lib/

selectors/

thread-selectors.js

10 lines

server/

src/

responders/

website-responders.js

7 lines

web/

redux/

redux-setup.js

4 lines

sidebar/

app-switcher.react.js

8 lines

Diff 10231

View Options

lib/selectors/thread-selectors.js

View Options

server/src/responders/website-responders.js

View Options

web/redux/redux-setup.js

View Options

web/sidebar/app-switcher.react.js

[web] Rename mostRecentReadThread to mostRecentlyReadThreadClosedPublicActions

Details

Diff Detail

Event Timeline

Revision ContentsChangeset List

Diff 10231

lib/selectors/thread-selectors.js

server/src/responders/website-responders.js

web/redux/redux-setup.js

web/sidebar/app-switcher.react.js

[web] Rename mostRecentReadThread to mostRecentlyReadThread
ClosedPublic
Actions

Revision Contents
Changeset List