Page MenuHomePhorge

[Breaking] Remove legacy trigger types
ClosedPublic

Authored by tusooa on Mon, Feb 16, 8:49 AM.

Details

Summary

Originally these types were added to notify callers of actions that it has been completed. Since we added the Promise-based store, this is no longer needed because callers can directly use then() to get the result of a dispatch. This is a better method than the original one, and no new trigger classes are added since then. However, the old ones remained. Now it is time to remove this legacy.

Also, the header kazvevents.hpp is renamed to kazv-triggers.hpp , and the types KazvEvent and KazvEventList are renamed KazvTrigger and KazvTriggerList to avoid confusion from the Event class, which represents a matrix event. Old names are kept as aliases to reduce incompatibility with older code.

Type: remove

Test Plan

Verify you can build and run kazv from this commit.

Diff Detail

Repository
rL libkazv
Lint
Lint Not Applicable
Unit
Tests Not Applicable