Page MenuHomePhabricator

[web][native] Extract common fetching logic
ClosedPublic

Authored by tomek on Aug 23 2024, 5:03 AM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Feb 3, 10:11 PM
Unknown Object (File)
Mon, Feb 3, 12:05 PM
Unknown Object (File)
Thu, Jan 30, 2:55 PM
Unknown Object (File)
Thu, Jan 30, 1:15 PM
Unknown Object (File)
Jan 15 2025, 11:10 AM
Unknown Object (File)
Dec 28 2024, 7:09 AM
Unknown Object (File)
Dec 28 2024, 7:09 AM
Unknown Object (File)
Dec 18 2024, 3:43 PM
Subscribers

Details

Summary

This logic is the same on both platforms and can be reused. Later in the stack I'm going to introduce the logic for thick thread fetching to this hook.

https://linear.app/comm/issue/ENG-8706/modify-message-lists-to-call-the-new-function

Depends on D13142

Test Plan

Tested on both native and web whether threads with less than 20 messages are displayed correctly and fetching all the messages, in batches, works for longer threads.

Diff Detail

Repository
rCOMM Comm
Branch
messages-pruning
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

tomek requested review of this revision.Aug 23 2024, 5:20 AM
kamil added inline comments.
lib/actions/message-actions.js
548

I would put this into lib/shared/message-utils.js but not feeling strongly

This revision is now accepted and ready to land.Aug 26 2024, 3:49 AM
This revision was automatically updated to reflect the committed changes.