We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
object-sender
The new object sender, during serialization, it's skipping all "not values", which includes empty strings or 0:
0
speckle-server/packages/objectsender/src/utils/Serializer.ts
Line 56 in 08d1bff
It is not by design right?
The .net serializer doesn't do it:
https://github.com/specklesystems/speckle-sharp-sdk/blob/ca350002ba17151585cc13564959362162294ae7/src/Speckle.Core/Serialisation/BaseObjectSerializer.cs#L321
Try to send whatever property that is an empty string or a zero :) it it is not showing up in speckle.
Empty strings and zero and, generally "empty" values should be pushed.
If applicable, please fill in the below details - they help a lot!
if (value === undefined || propKey === 'id' || propKey.startsWith('_')) continue
The text was updated successfully, but these errors were encountered:
Successfully merging a pull request may close this issue.
Prerequisites
What package are you referring to?
object-sender
Describe the bug
The new object sender, during serialization, it's skipping all "not values", which includes empty strings or
0
:speckle-server/packages/objectsender/src/utils/Serializer.ts
Line 56 in 08d1bff
It is not by design right?
The .net serializer doesn't do it:
https://github.com/specklesystems/speckle-sharp-sdk/blob/ca350002ba17151585cc13564959362162294ae7/src/Speckle.Core/Serialisation/BaseObjectSerializer.cs#L321
To Reproduce
Try to send whatever property that is an empty string or a zero :) it it is not showing up in speckle.
Expected behavior
Empty strings and zero and, generally "empty" values should be pushed.
Screenshots
System Info
If applicable, please fill in the below details - they help a lot!
Desktop (please complete the following information):
Smartphone (please complete the following information):
Failure Logs
Additional context
Proposed Solution (if any)
Optional: Affected Projects
The text was updated successfully, but these errors were encountered: