Page MenuHomePhabricator

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

Authored by tomek on Sep 2 2024, 7:41 AM.
Tags
None
Referenced Files
F3197224: D13219.diff
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)
Tue, Oct 22, 6:29 PM
Unknown Object (File)
Fri, Oct 18, 5:16 PM
Unknown Object (File)
Fri, Oct 18, 5:16 PM
Unknown Object (File)
Fri, Oct 18, 5:16 PM
Unknown Object (File)
Fri, Oct 18, 5:16 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
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