Page MenuHomePhorge
Differential D237

Add purging event support
ClosedPublic
Actions

Authored by tusooa on Oct 1 2025, 8:37 PM.

Details

Reviewers
nannanko
Group Reviewers
O1: the Kazv Project
Commits
rL9fc4b3e0da9d: Add purging event support
Summary

This adds function to purge room events.

Type: add

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

Test Plan

Verify unit tests pass.

Diff Detail

Repository
rL libkazv
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

tusooa created this revision.Oct 1 2025, 8:37 PM
Owners added a reviewer: O1: the Kazv Project.Oct 1 2025, 8:37 PM
Harbormaster completed remote builds in B451: Diff 707.Oct 1 2025, 9:08 PM
tusooa requested review of this revision.Oct 1 2025, 9:08 PM
nannanko subscribed.Oct 4 2025, 7:44 PM
nannanko added inline comments.
src/client/room/room-model.hpp
348
tusooa added inline comments.Oct 5 2025, 7:38 PM
src/client/room/room-model.hpp
348

iff is correct. it means if and only if. using if here will make the return condition incomplete because it does not specify what to return when the invariants are not satisfied.

nannanko accepted this revision.Oct 5 2025, 10:52 PM
This revision is now accepted and ready to land.Oct 5 2025, 10:52 PM
tusooa updated this revision to Diff 717.Oct 7 2025, 2:44 PM

Add tests for purging

tusooa marked an inline comment as done.Oct 7 2025, 2:45 PM
Harbormaster completed remote builds in B460: Diff 717.Oct 7 2025, 3:18 PM
tusooa updated this revision to Diff 719.Oct 8 2025, 3:14 PM

Change action to allow purging multiple rooms

tusooa edited the test plan for this revision. (Show Details)Oct 8 2025, 3:18 PM
Harbormaster completed remote builds in B461: Diff 719.Oct 8 2025, 3:44 PM
tusooa updated this revision to Diff 734.Oct 13 2025, 5:40 PM

Rebase

Harbormaster completed remote builds in B471: Diff 734.Oct 13 2025, 6:42 PM
Closed by commit rL9fc4b3e0da9d: Add purging event support. · Explain WhySun, Dec 14, 6:00 PM
This revision was automatically updated to reflect the committed changes.
tusooa added a commit: rL9fc4b3e0da9d: Add purging event support.

Revision Contents
Changeset List

Diff 766

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.hpp

Loading...
View Options

src/client/client.cpp

Loading...
View Options

src/client/clientfwd.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/action-mock-utils.hpp

Loading...
View Options

src/tests/client/room/purge-test.cpp

Loading...