Skip to content

Conversation

@rafaeltonholo
Copy link
Member

Part of #9497.

This PR introduces a StateMachine class, which allows us to define a set of states with transitions triggered by events. This will help in the refactor of the Message List, improving the state handling.

@rafaeltonholo rafaeltonholo requested a review from a team as a code owner November 28, 2025 20:41
@rafaeltonholo rafaeltonholo force-pushed the feat/9497/add-state-machine branch 4 times, most recently from 3203e8a to b6bb620 Compare December 1, 2025 17:35
@wmontwe wmontwe added merge block: soft freeze PR to main is blocked: risky code or feature flag enablement must wait until soft freeze lifts. and removed merge block: soft freeze PR to main is blocked: risky code or feature flag enablement must wait until soft freeze lifts. labels Dec 4, 2025
@rafaeltonholo rafaeltonholo force-pushed the feat/9497/add-state-machine branch from b6bb620 to 6d361a5 Compare December 10, 2025 14:25
@rafaeltonholo rafaeltonholo force-pushed the feat/9497/add-state-machine branch from 6d361a5 to 96ed140 Compare December 12, 2025 15:56
@rafaeltonholo rafaeltonholo force-pushed the feat/9497/add-state-machine branch from 96ed140 to 26c96b2 Compare January 5, 2026 18:51
@rafaeltonholo rafaeltonholo force-pushed the feat/9497/add-state-machine branch from 26c96b2 to 703e021 Compare January 5, 2026 20:29
Copy link
Member

@wmontwe wmontwe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me from a high-level perspective. I still need to exercise an implementation, but that shouldn’t block these changes.

@rafaeltonholo rafaeltonholo merged commit 59ffddf into thunderbird:main Jan 6, 2026
12 checks passed
@thunderbird-botmobile thunderbird-botmobile bot added this to the Thunderbird 17 milestone Jan 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants