Page MenuHomePhabricator

[lib] Include `timestamps` inside `CreateThickRawThreadInfoInput`
ClosedPublic

Authored by tomek on Sep 2 2024, 7:41 AM.
Tags
None
Referenced Files
F3342137: D13219.diff
Fri, Nov 22, 12:39 AM
Unknown Object (File)
Tue, Nov 12, 2:21 PM
Unknown Object (File)
Sat, Nov 9, 11:40 PM
Unknown Object (File)
Sat, Nov 9, 12:40 PM
Unknown Object (File)
Sat, Nov 9, 7:47 AM
Unknown Object (File)
Mon, Nov 4, 6:27 PM
Unknown Object (File)
Sun, Nov 3, 2:34 AM
Unknown Object (File)
Oct 22 2024, 6:29 PM
Subscribers

Details

Summary

When a user is added to a thread, they should get the correct timestamps instead of assuming that thread creation time or operation time are correct.

https://linear.app/comm/issue/ENG-9124/send-timestamps-in-existingthreaddetails

Depends on D13218

Test Plan

Create an operation with the creation of a new thread. Check if the timestamps are correct.

Diff Detail

Repository
rCOMM Comm
Branch
timestamps
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

tomek requested review of this revision.Sep 2 2024, 7:58 AM
This revision is now accepted and ready to land.Sep 3 2024, 3:59 AM

Pass timestamps directly to createThickRawThreadInfo