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.

Sign up for GitHub

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

Jump to bottom

JSON serialization of "null" now assigns a correct empty value for every possible data type #666

Merged
merged 7 commits into from
Mar 6, 2023
Merged

JSON serialization of "null" now assigns a correct empty value for every possible data type #666

merged 7 commits into from
Mar 6, 2023

Conversation

tomas-sexenian
Copy link
Contributor

No description provided.

@genexusbot
Copy link
Collaborator

Cherry pick to beta success

@genexusbot
Copy link
Collaborator

Cherry pick to beta success

@tomas-sexenian tomas-sexenian changed the title FromJson now assigns empty string instead of "null" in case of null input JSON serialization of "null" now assigns a correct empty value for every possible data type Feb 7, 2023
@genexusbot
Copy link
Collaborator

Cherry pick to beta failed, 1 conflicted file in commit 693d0f4
  • java/src/test/java/com/genexus/TestCommonUtil.java

@tomas-sexenian
Copy link
Contributor Author

Cherry-pick conflict has been manually resolved

@genexusbot
Copy link
Collaborator

Cherry pick to beta failed, 1 conflicted file in commit 693d0f4
  • java/src/test/java/com/genexus/TestCommonUtil.java

ggallotti
ggallotti previously approved these changes Feb 16, 2023
View reviewed changes
@tomas-sexenian tomas-sexenian removed the request for review from iroqueta February 16, 2023 13:15
@genexusbot
Copy link
Collaborator

Cherry pick to beta failed, 1 conflicted file in commit 693d0f4
  • java/src/test/java/com/genexus/TestCommonUtil.java

@tomas-sexenian tomas-sexenian requested a review from iroqueta March 6, 2023 12:44
@tomas-sexenian tomas-sexenian merged commit 5c423f4 into master Mar 6, 2023
@tomas-sexenian tomas-sexenian deleted the IncorrectNullString branch March 6, 2023 17:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@iroqueta iroqueta iroqueta approved these changes

@ggallotti ggallotti ggallotti left review comments

Assignees
No one assigned
Labels
bot closed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@tomas-sexenian @genexusbot @ggallotti @iroqueta @cinthamo