Skip to content

Conversation

@lilgallon
Copy link
Member

@lilgallon lilgallon commented Dec 23, 2025

closes #184

-> maintenance of kotlinx.serializers will be handled in another independent project (which has nothing to do with ocpi)

Added deprecation notice (the IDE should automatically detect and suggest to fix it):

Only for users explicitly calling OCPI serializers:

import com.izivia.ocpi.toolkit.integrations.kotlinx.serialization.generated.generatedSerializersModule
// will be changed to:
import com.izivia.ocpi.toolkit.integrations.kotlinx.serialization.ocpiSerializersModule

@lilgallon lilgallon force-pushed the feat/#184-konstruct branch 3 times, most recently from 3c63b26 to dfe90e7 Compare December 29, 2025 21:26
@lilgallon
Copy link
Member Author

Tested internally on a project using kotlinx.serialization support ✅

No backward incompatibility. The field becomes deprecated and IntelliJ automatically detects what should be changed. With the old and the new field, everything works as expected

@sonarqubecloud
Copy link

@lilgallon lilgallon merged commit 518dd56 into main Dec 29, 2025
2 checks passed
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.

Use konstruct lib instead of custom serializer generator

2 participants