Skip to content

JAVA-5736 Add bsonNamingStrategy option to support snake_case #1627

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 3 commits into from
Mar 19, 2025
Merged

Conversation

nhachicha
Copy link
Contributor

Fixes Java-5736
This PR adds some refactoring to #1589

leesungbin and others added 2 commits February 6, 2025 17:08
… 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
@nhachicha nhachicha requested a review from rozza February 7, 2025 12:37
@nhachicha nhachicha self-assigned this Feb 7, 2025
@nhachicha nhachicha changed the title Java 5736 Add bsonNamingStrategy option to support snake_case Feb 7, 2025
@nhachicha nhachicha changed the title Add bsonNamingStrategy option to support snake_case JAVA-5736 Add bsonNamingStrategy option to support snake_case Feb 7, 2025
Copy link
Member

@rozza rozza left a 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.

@rozza rozza merged commit 48fdf2a into main Mar 19, 2025
19 of 20 checks passed
@rozza rozza deleted the JAVA-5736 branch March 19, 2025 14:16
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants