Add support for kotlinx-datetime serializers mapping to BSON #1462
Conversation
- Add kotlinx-datetime serializers that map to BSON as the expected types. - Make it easily configurable via `@Contextual` annotation. JAVA-5330
JAVA-5330
bson-kotlinx/build.gradle.kts
Outdated
| @@ -45,6 +45,7 @@ dependencies { | |||
|
|
|||
| implementation(platform("org.jetbrains.kotlinx:kotlinx-serialization-bom:1.5.0")) | |||
| implementation("org.jetbrains.kotlinx:kotlinx-serialization-core") | |||
| implementation("org.jetbrains.kotlinx:kotlinx-datetime:0.4.0") | |||
There was a problem hiding this comment.
This version is compatible with Kotlin versions 1.5 and later. The next version 0.5.0 requires Kotlin 1.9 or higher, as indicated by this commit:
There was a problem hiding this comment.
We can't take a hard dependency on this library - so this will need refactoring to make it optional. #1459 adds a check for kotlinx-json support, you could use that style to add an if statement for the defaultSerializersModule
There was a problem hiding this comment.
I have refactored it to make the library optional, using the approach outlined in #1459 for checking kotlinx-datetime support. Thanks!
bson-kotlinx/build.gradle.kts
Outdated
| @@ -45,6 +45,7 @@ dependencies { | |||
|
|
|||
| implementation(platform("org.jetbrains.kotlinx:kotlinx-serialization-bom:1.5.0")) | |||
| implementation("org.jetbrains.kotlinx:kotlinx-serialization-core") | |||
| implementation("org.jetbrains.kotlinx:kotlinx-datetime:0.4.0") | |||
There was a problem hiding this comment.
We can't take a hard dependency on this library - so this will need refactoring to make it optional. #1459 adds a check for kotlinx-json support, you could use that style to add an if statement for the defaultSerializersModule
bson-kotlinx/src/main/kotlin/org/bson/codecs/kotlinx/DateTimeSerializers.kt
Show resolved
Hide resolved
bson-kotlinx/src/main/kotlin/org/bson/codecs/kotlinx/DateTimeSerializers.kt
Outdated
Show resolved
Hide resolved
…erializers.kt Co-authored-by: Ross Lawley <[email protected]>
JAVA-5330
bson-kotlinx/src/main/kotlin/org/bson/codecs/kotlinx/DateTimeSerializers.kt
Outdated
Show resolved
Hide resolved
bson-kotlinx/src/main/kotlin/org/bson/codecs/kotlinx/DateTimeSerializers.kt
Outdated
Show resolved
Hide resolved
bson-kotlinx/src/main/kotlin/org/bson/codecs/kotlinx/DateTimeSerializers.kt
Outdated
Show resolved
Hide resolved
@Contextualannotation.JAVA-5330