liveview-native / liveview-native-core

Provides core language-agnostic functionality for LiveView Native across platforms
MIT License
146 stars 10 forks source link

Hang in document change handler. #226

Closed mobile-bungalow closed 1 week ago

mobile-bungalow commented 1 week ago

handle implementations should be able to run assuming no other locks to the document are held. This means we need to collect a vec of patches before passing them one by one to the handler code, instead of doing it inside the merge_fragment_json function, otherwise this potentially locks the client.

previously this would have just mutated the document while iterating.

mobile-bungalow commented 1 week ago

fixed in #229