Adding Temporal

[kpartlow]

Conversion Tests
LocalDate conversions
ZonedDateTime Updates

[jdereg]

Does this API need to use: options.getSourceZoneIdForLocalDates()

[kpartlow]

The epoch types don't because they are in the epochMillis

…

On Tue, Jan 23, 2024 at 10:10 AM John DeRegnaucourt < ***@***.***> wrote: ***@***.**** commented on this pull request. ------------------------------ In src/main/java/com/cedarsoftware/util/convert/LocalDateConversion.java <#83 (comment)>: > + + static Instant toInstant(Object fromInstance, ConverterOptions options) { + return toZonedDateTime(fromInstance, options).toInstant(); + } + + static long toLong(Object fromInstance, ConverterOptions options) { + return toInstant(fromInstance, options).toEpochMilli(); + } + + static LocalDateTime toLocalDateTime(Object fromInstance, Converter converter, ConverterOptions options) { + return toZonedDateTime(fromInstance, options).toLocalDateTime(); + } + + static ZonedDateTime toZonedDateTime(Object fromInstance, Converter converter, ConverterOptions options) { + return toZonedDateTime(fromInstance, options).withZoneSameInstant(options.getZoneId()); + } Does this API need to use: options.getSourceZoneIdForLocalDates() — Reply to this email directly, view it on GitHub <#83 (review)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ABPE7YXTHG2OBVZSGEDTKCTYP7HFDAVCNFSM6AAAAABCGOPQAWVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTQMZZGAZTAMJYHA> . You are receiving this because you authored the thread.Message ID: ***@***.***>