Page MenuHomePhorge
Differential D233

Add functionality to load events
AcceptedPublic
Actions

Authored by tusooa on Mon, Sep 29, 4:16 PM.
Tags
None
Referenced Files
F8641960: D233.1761623508.diff
Sun, Oct 26, 8:51 PM
F8641959: D233.1761623507.diff
Sun, Oct 26, 8:51 PM
F8641958: D233.1761623506.diff
Sun, Oct 26, 8:51 PM
F8641957: D233.1761623505.diff
Sun, Oct 26, 8:51 PM
F8641242: D233.1761617634.diff
Sun, Oct 26, 7:13 PM
Subscribers
None

Details

Reviewers
nannanko
Group Reviewers
O1: the Kazv Project
Summary

This adds actions for loading into the lager store.

Type: add

CCBUG: https://iron.lily-is.land/T165
BUG: https://iron.lily-is.land/T26

Test Plan

Build kazv against this and verify everything functions as usual, paying special attention to: (1) event relationships (verify reply and edits are displayed properly), (2) timeline gaps (verify you can paginate back as normal).

Diff Detail

Repository
rL libkazv
Branch
tusooa/stacked
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 449
Build 881: GitLab CI for libkazv
Build 880: arc lint + arc unit

Event Timeline

tusooa created this revision.Mon, Sep 29, 4:16 PM
Owners added a reviewer: O1: the Kazv Project.Mon, Sep 29, 4:16 PM
Harbormaster returned this revision to the author for changes because remote builds failed.Mon, Sep 29, 4:40 PM
Harbormaster failed remote builds in B440: Diff 694!
tusooa updated this revision to Diff 705.Wed, Oct 1, 3:04 PM

Remove purge event action for later

tusooa retitled this revision from Add functionality to load and purge events to Add functionality to load events.Wed, Oct 1, 3:13 PM
tusooa edited the summary of this revision. (Show Details)
tusooa edited the test plan for this revision. (Show Details)
tusooa updated this revision to Diff 706.Wed, Oct 1, 3:14 PM

Remove remaining purge action in client-model.hpp

Harbormaster completed remote builds in B449: Diff 705.Wed, Oct 1, 3:42 PM
Harbormaster completed remote builds in B450: Diff 706.Wed, Oct 1, 3:55 PM
tusooa requested review of this revision.Wed, Oct 1, 3:55 PM
tusooa edited the summary of this revision. (Show Details)Wed, Oct 1, 8:33 PM
nannanko accepted this revision.Sat, Oct 4, 7:38 PM
This revision is now accepted and ready to land.Sat, Oct 4, 7:38 PM
tusooa updated this revision to Diff 716.Tue, Oct 7, 2:41 PM

remove PurgeEventsAction

Harbormaster completed remote builds in B459: Diff 716.Tue, Oct 7, 3:13 PM

Revision Contents
Changeset List

Diff 705

View Options

src/client/CMakeLists.txt

Loading...
View Options

src/client/actions/storage.hpp

Loading...
View Options

src/client/actions/storage.cpp

Loading...
View Options

src/client/client-model.hpp

Loading...
View Options

src/client/client-model.cpp

Loading...
View Options

src/client/clientfwd.hpp

Loading...
View Options

src/client/clientutil.hpp

Loading...
View Options

src/client/room/room-model.hpp

Loading...
View Options

src/client/room/room-model.cpp

Loading...
View Options

src/tests/CMakeLists.txt

Loading...
View Options

src/tests/client/storage-actions-test.cpp

Loading...
View Options

src/tests/client/util-test.cpp

Loading...