Page MenuHomePhabricator

Conpherence - fix disappearing older messages bug
ClosedPublic

Authored by btrahan on May 14 2015, 6:09 PM.
Tags
None
Referenced Files
F19100227: D12844.id.diff
Thu, Dec 4, 10:35 PM
F19091332: D12844.diff
Wed, Dec 3, 2:27 PM
F18832785: D12844.id.diff
Oct 25 2025, 11:02 PM
F18828134: D12844.diff
Oct 24 2025, 2:31 PM
F18778482: D12844.id30893.diff
Oct 11 2025, 9:38 AM
F18764601: D12844.id.diff
Oct 7 2025, 6:49 AM
F18693678: D12844.id30894.diff
Sep 27 2025, 1:40 AM
F18640342: D12844.diff
Sep 18 2025, 3:40 PM
Subscribers

Details

Summary

Fixes T8194. If you loaded a conpherence directly via a URI like /Z1 or /conpherence/1/ we didn't correctly cache the loaded transactions such that sending a message would only show that message and subsequent messages.

Fix is to build the cache properly in this case. Note this codepath is different because the server sends back more of the thread data in the initial request.

Test Plan

loaded a conpherence directly, sent messages, paged back, everything kept showing!

Diff Detail

Repository
rP Phabricator
Branch
T8194
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 5985
Build 6005: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

btrahan retitled this revision from to Conpherence - fix disappearing older messages bug.
btrahan updated this object.
btrahan edited the test plan for this revision. (Show Details)
btrahan added a reviewer: epriestley.
epriestley edited edge metadata.
This revision is now accepted and ready to land.May 14 2015, 6:13 PM
This revision was automatically updated to reflect the committed changes.