-
Notifications
You must be signed in to change notification settings - Fork 37
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
Upgrade breaks Interaction serialization with native variadic arguments #306
Comments
Hi @michavie, Thank you for raising the issue 🙏 Do you think you can help us reproduce the issue, by altering the following test case (the one below passes)?
Thank you! |
I could reproduce the error in the test results by not providing any values, as in spreading an empty array: Would make sense as Sorry for the initial sloppy example Submitted a quick PR here: #307 |
Fixed by your PR, #307. Thank you :) |
Upgrading from
12.1.0
breaks transactions using native variadic arguments, with this error:Worked before and fails after upgrade for Interactions:
Using
VariadicValue
is a temporary fix, but ideally, native values would keep workingLikely caused by this change here: 973c823
The text was updated successfully, but these errors were encountered: