Skip to content

Conversation

pcarranzav
Copy link
Member

@pcarranzav pcarranzav commented Jan 30, 2025

We were always passing undefined as the previous state, so the applyEvent was failing when there was more than 1 event.

For some reason this broke type inference, so had to add some explicit types here.

@nikgraf
Copy link
Collaborator

nikgraf commented Jan 31, 2025

good catch! I was sure I double-checked for this when I added it, but seems like I didn't 🤦

@nikgraf nikgraf merged commit 1d018f1 into main Jan 31, 2025
4 checks passed
@nikgraf nikgraf deleted the pcv/fix-apply-event branch January 31, 2025 09:36
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