Page MenuHomePhabricator

Conpherence - fix disappearing older messages bug
ClosedPublic

Authored by btrahan on May 14 2015, 6:09 PM.
Tags
None
Referenced Files
F13182303: D12844.diff
Thu, May 9, 10:45 PM
F13178403: D12844.diff
Wed, May 8, 8:23 PM
Unknown Object (File)
Thu, May 2, 2:38 PM
Unknown Object (File)
Thu, Apr 25, 2:16 AM
Unknown Object (File)
Apr 1 2024, 12:45 PM
Unknown Object (File)
Mar 31 2024, 1:40 AM
Unknown Object (File)
Mar 30 2024, 9:29 PM
Unknown Object (File)
Mar 30 2024, 9:16 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.