Skip to content

Refactor BroadcastSet to follow consistent entity definition patterns#3

Merged
arminsabouri merged 1 commit intopayjoin:masterfrom
0xZaddyy:cleanup-broadcast-set
Feb 4, 2026
Merged

Refactor BroadcastSet to follow consistent entity definition patterns#3
arminsabouri merged 1 commit intopayjoin:masterfrom
0xZaddyy:cleanup-broadcast-set

Conversation

@0xZaddyy
Copy link
Contributor

@0xZaddyy 0xZaddyy commented Feb 4, 2026

This PR refactors the codebase to follow consistent definition patterns by consolidating entity to use the unified define_entity! macro

Summary

  • Refactor BroadcastSet entity to use unified define_entity! macro for consistency with other entities
  • Rename BroadcastSetData enum to BroadcastSetType and integrate into entity system
  • Add proper mutable handle generation via define_entity_handle_mut! macro

@0xZaddyy 0xZaddyy changed the title Cleanup broadcast set Refactor BroadcastSet to follow consistent entity definition patterns Feb 4, 2026
@arminsabouri
Copy link
Collaborator

@0xZaddyy can you please squash your two commits together

Copy link
Collaborator

@arminsabouri arminsabouri left a comment

Choose a reason for hiding this comment

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

Cack
just needs commits to be squashed

@0xZaddyy 0xZaddyy force-pushed the cleanup-broadcast-set branch from 3452555 to 33b677e Compare February 4, 2026 16:58
Copy link
Collaborator

@arminsabouri arminsabouri left a comment

Choose a reason for hiding this comment

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

re-ACK

@arminsabouri arminsabouri merged commit 313bd3c into payjoin:master Feb 4, 2026
1 check passed
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