-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
JAVA-5736 Add bsonNamingStrategy option to support snake_case #1627
base: main
Are you sure you want to change the base?
Conversation
… strategy (#1589) * Add bsonNamingStrategy option to support snake_case naming strategy * Fix case converting logic with caching * Refactor imports in KotlinSerializerCodecTest.kt * Reimport java.util.Base64
…ith snake case - Fixing formatting
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there a way to hook directly to use serialNameForJson
then this could support all strategies for naming.
If not then we should add a section to THIRD-PARTY-NOTICES
for the convertCamelCase
method.
println("Deserializing: ${value.toJson()}") | ||
val codec = KotlinSerializerCodec.create(T::class, serializersModule, configuration)!! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
println("Deserializing: ${value.toJson()}") | |
val codec = KotlinSerializerCodec.create(T::class, serializersModule, configuration)!! | |
val codec = KotlinSerializerCodec.create(T::class, serializersModule, configuration)!! |
) | ||
|
||
public enum class BsonNamingStrategy { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does this need docs?
Fixes Java-5736
This PR adds some refactoring to #1589