Skip to content

Conversation

GnaneshKunal
Copy link
Contributor

@GnaneshKunal GnaneshKunal commented Oct 22, 2024

Fixes: #2667

image


I raised a PR for the library repository and patched RedisInsight to work on the tip of the fix.

Library PR: asherkin/java-object-serialization#1

By the way, the library was last updated 3 years ago.

@GnaneshKunal GnaneshKunal self-assigned this Oct 22, 2024
@GnaneshKunal GnaneshKunal force-pushed the bugfix/fe/java-ser-util-date branch from 012c299 to 1927a93 Compare October 31, 2024 13:43
@GnaneshKunal GnaneshKunal requested review from egor-zalenski and rsergeenko and removed request for egor-zalenski October 31, 2024 13:45
rsergeenko
rsergeenko previously approved these changes Nov 1, 2024
rsergeenko
rsergeenko previously approved these changes Nov 18, 2024
Copy link
Contributor

@rsergeenko rsergeenko left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm. It would be nice to have more tests for dates (negative, positive, some corner cases)

# Conflicts:
#	tests/e2e/tests/web/regression/browser/formatters.e2e.ts
#	yarn.lock
@mariasergeenko mariasergeenko merged commit 22dcf40 into main Nov 21, 2024
3 checks passed
@mariasergeenko mariasergeenko deleted the bugfix/fe/java-ser-util-date branch November 21, 2024 10:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature Request]:Java serialized support java.util.Date or java.time.LocalDateTime ...
3 participants