Skip to content

introduce useRoom() and createRoomContext()#6

Merged
endel merged 1 commit intomainfrom
use-room
Feb 25, 2026
Merged

introduce useRoom() and createRoomContext()#6
endel merged 1 commit intomainfrom
use-room

Conversation

@endel
Copy link
Member

@endel endel commented Feb 25, 2026 •
edited
Loading

This was referenced Feb 25, 2026
Merged
Merged
@endel endel merged commit daa471b into main Feb 25, 2026
1 check passed
@endel
Copy link
Member Author

endel commented Feb 25, 2026

I hope it's not too early, I'm merging this so we can get feedback for real usage and course-correct if needed 🙌

@endel endel deleted the use-room branch February 25, 2026 20:29
@FTWinston
Copy link
Collaborator

FTWinston commented Feb 25, 2026

Ha, you beat me to it 😄

I was going to leave this feedback:

  • I think index.ts should be exporting createRoomContext ... it isn't currently
  • Renaming useRoomState's T template parameter to State is a good move ... but then I think U should be renamed to T.

All the rest looks great to me!

@endel endel mentioned this pull request Feb 25, 2026
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@endel @FTWinston