-
Notifications
You must be signed in to change notification settings - Fork 74
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Reapply "Defer serializing event args to JSON (#2218)" (#2239) #2262
Conversation
@@ -64,7 +65,7 @@ class ViewRecyclingTester( | |||
container = widgetContainer, | |||
factory = widgetProtocolFactory, | |||
eventSink = { event -> | |||
guestAdapter.sendEvent(event) | |||
guestAdapter.sendEvent(event.toProtocol(Json.Default)) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This wasn’t in the original PR, because this test is new!
06d2e9c
to
cd4bad5
Compare
I’m set up to test it end-to-end now and I’ll confirm it works before I merge... |
Thanks! As a reminder, the bug would only manifest after scrolling down a few pages (to force some recycling to occur) |
Yep, confirmed the bug is fixed. |
This reverts commit 5f8ca01.
CHANGELOG.md
's "Unreleased" section has been updated, if applicable.