Skip to content

onchain allocation: CreateSubscribeUser atomic+onchain allocation#3176

Merged
elitegreg merged 4 commits intomainfrom
gm/user_subscribe_refactor
Mar 9, 2026
Merged

onchain allocation: CreateSubscribeUser atomic+onchain allocation#3176
elitegreg merged 4 commits intomainfrom
gm/user_subscribe_refactor

Conversation

@elitegreg
Copy link
Contributor

@elitegreg elitegreg commented Mar 5, 2026

Closes #3127

Summary of Changes

  • Refactors CreateUser/MulticastSubscribe to extract the business logic so that CreateSubscribeUser can be built from those 2 pieces.
  • Updates CreateSubscribeUser to use shared CreateUser/MulticastSubscribe logic as well as enable onchain allocation

Testing Verification

  • New tests added
  • Existing tests pass

@elitegreg elitegreg force-pushed the gm/user_subscribe_refactor branch 2 times, most recently from 3b1c11e to 135401c Compare March 5, 2026 04:12
@elitegreg elitegreg marked this pull request as ready for review March 5, 2026 21:13
@elitegreg elitegreg requested a review from vihu March 5, 2026 21:13
@elitegreg elitegreg force-pushed the gm/user_subscribe_refactor branch 2 times, most recently from 07be121 to e9b5cb3 Compare March 6, 2026 16:59
@elitegreg elitegreg enabled auto-merge (squash) March 6, 2026 17:19
@elitegreg elitegreg force-pushed the gm/user_subscribe_refactor branch from 34e42d4 to f670854 Compare March 9, 2026 15:17
@elitegreg elitegreg merged commit 997f5dc into main Mar 9, 2026
30 checks passed
@elitegreg elitegreg deleted the gm/user_subscribe_refactor branch March 9, 2026 16:19
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.

Phase 2: Modify CreateSubscribeUser for atomic create+allocate+activate

2 participants