Skip to content
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

fix(go): enums inside structs are not properly serialized #2636

Merged
merged 7 commits into from
Mar 2, 2021

Conversation

eladb
Copy link
Contributor

@eladb eladb commented Mar 2, 2021

Serializes struct values according to the jsii protocol, wrapping
the data in a $jsii.struct object recording both the struct's
fully qualified type name, and key-value pairs.

This includes recursively transforming field values, so the
behavior is now correct.


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

This is only a failing test for now
@mergify mergify bot added the contribution/core This is a PR that came from AWS. label Mar 2, 2021
@RomainMuller RomainMuller marked this pull request as ready for review March 2, 2021 15:07
@eladb
Copy link
Contributor Author

eladb commented Mar 2, 2021

LGTM

@RomainMuller RomainMuller self-assigned this Mar 2, 2021
@RomainMuller RomainMuller merged commit 19cbd25 into main Mar 2, 2021
@RomainMuller RomainMuller deleted the fix/enums-in-structs branch March 2, 2021 16:35
This was referenced Mar 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
contribution/core This is a PR that came from AWS.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants