Page MenuHomePhabricator
Differential D13214

[lib] Introduce thread timestamps on the JS side
ClosedPublic
Actions

Authored by tomek on Aug 30 2024, 9:07 AM.
Tags
None
Referenced Files
F3197248: D13214.id43930.diff
Sat, Nov 9, 7:50 AM
F3197229: D13214.id43820.diff
Sat, Nov 9, 7:49 AM
Unknown Object (File)
Sun, Nov 3, 9:26 PM
Unknown Object (File)
Sun, Nov 3, 9:26 PM
Unknown Object (File)
Sun, Nov 3, 9:25 PM
Unknown Object (File)
Sun, Nov 3, 9:25 PM
Unknown Object (File)
Sun, Nov 3, 2:34 AM
Unknown Object (File)
Sat, Oct 26, 3:41 AM
View All 60 Files
Subscribers
ashoat

Details

Reviewers
kamil
inka
Commits
rCOMMf72ecb2ea151: [lib] Introduce thread timestamps on the JS side
Summary

Create a new property with the timestamps. All the new thick threads will have it. We don't really need to have a migration introducing them, because all the thick threads we have were created by code on dev environments.

https://linear.app/comm/issue/ENG-8751/add-a-timestamps-property-to-a-thick-thread-info

Depends on D13211

Test Plan

On native and web, create a new thick thread, check if it contains the timestamps. Restart the apps and check if the timestamps are still there.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

tomek created this revision.Aug 30 2024, 9:07 AM
Herald added a subscriber: ashoat. · View Herald TranscriptAug 30 2024, 9:07 AM
tomek added a parent revision: D13211: [native][web] Add a timestamp column to threads table.Aug 30 2024, 9:08 AM
tomek edited the summary of this revision. (Show Details)
tomek updated this revision to Diff 43820.Aug 30 2024, 9:29 AM

Remove replies count timestamp - the replies count is based on the messages which means it doesn't need a timestamp.

Harbormaster completed remote builds in B31375: Diff 43819.Aug 30 2024, 9:40 AM
Harbormaster completed remote builds in B31376: Diff 43820.Aug 30 2024, 9:47 AM
tomek requested review of this revision.Aug 30 2024, 9:47 AM
tomek added a child revision: D13215: [lib] Check the timestamps when changing thread settings.Aug 30 2024, 10:17 AM
tomek updated this revision to Diff 43827.Sep 2 2024, 7:20 AM

Rename

Harbormaster completed remote builds in B31382: Diff 43827.Sep 2 2024, 7:37 AM
inka accepted this revision.Sep 3 2024, 3:06 AM
This revision is now accepted and ready to land.Sep 3 2024, 3:06 AM
inka added inline comments.Sep 3 2024, 3:13 AM
lib/types/thread-types.js
152 ↗(On Diff #43827)

What is this representing?

tomek added inline comments.Sep 5 2024, 1:21 AM
lib/types/thread-types.js
152 ↗(On Diff #43827)

It represents the last change of the unread status.

Closed by commit rCOMMf72ecb2ea151: [lib] Introduce thread timestamps on the JS side (authored by tomek). · Explain WhySep 6 2024, 8:10 AM
This revision was automatically updated to reflect the committed changes.
tomek added a commit: rCOMMf72ecb2ea151: [lib] Introduce thread timestamps on the JS side.

Revision Contents
Changeset List

PathSize
lib/
shared/
dm-ops/
9 lines
24 lines
types/
35 lines
utils/
14 lines
1 line

Diff 43930

View Options

lib/shared/dm-ops/create-thread-spec.js

Loading...
View Options

lib/shared/thread-utils.js

Loading...
View Options

lib/types/thread-types.js

Loading...
View Options

lib/utils/thread-ops-utils.js

Loading...
View Options

lib/utils/thread-ops-utils.test.js

Loading...