Johann Pardanaud

Results 58 comments of Johann Pardanaud

I'm not sure I want to see the payload details to leak into [AkkurateConfig](https://akkurate.dev/api/akkurate-ktor-server/dev.nesk.akkurate.ktor.server/-akkurate-config/index.html?query=class%20AkkurateConfig) because it would imply a tight coupling between this class and the [ProblemDetailsMessage](https://akkurate.dev/api/akkurate-ktor-server/dev.nesk.akkurate.ktor.server/-problem-details-message/index.html?query=class%20ProblemDetailsMessage(val%20status:%20Int,%20val%20fields:%20Set%3CConstraintViolation%3E)) one. However, this...

This still brings some tight coupling, however, we might find a solution with extension functions. I'll think about it.

This is clearly an issue I want to address! I think your idea of metadata is good. We could even add a dedicated `code` property to `ConstraintViolation`, because this is...

You're right, I thought it would be possible for the implementation to override the default value provided by the interface, but it's not. The reason I want an interface here...

This doesn't seem to be an option because the default value is part of the method signature. [Here's an example:](https://akkurate.dev/api/akkurate-core/dev.nesk.akkurate/-validator/-companion/index.html) It would be weird for the users (and the maintainers)...

> After checking [referenced kotlin issue](https://youtrack.jetbrains.com/issue/KT-59493), it seems it is not an actual root of this problem. Actual referenced issue should be [this one](https://youtrack.jetbrains.com/issue/KT-62543). Yeah, I'm the author of both...

> When changing method like this > > ![image](https://private-user-images.githubusercontent.com/3338261/397011869-f525beba-c4c5-4afd-8349-c20acfe8a179.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MzQ2NDQyODYsIm5iZiI6MTczNDY0Mzk4NiwicGF0aCI6Ii8zMzM4MjYxLzM5NzAxMTg2OS1mNTI1YmViYS1jNGM1LTRhZmQtODM0OS1jMjBhY2ZlOGExNzkucG5nP1gtQW16LUFsZ29yaXRobT1BV1M0LUhNQUMtU0hBMjU2JlgtQW16LUNyZWRlbnRpYWw9QUtJQVZDT0RZTFNBNTNQUUs0WkElMkYyMDI0MTIxOSUyRnVzLWVhc3QtMSUyRnMzJTJGYXdzNF9yZXF1ZXN0JlgtQW16LURhdGU9MjAyNDEyMTlUMjEzMzA2WiZYLUFtei1FeHBpcmVzPTMwMCZYLUFtei1TaWduYXR1cmU9ZGFlYzI2MDBjMzBjOTYxOTIzNDA4NzNkYmQ0MDkzOGIzZDgxNTQ0ZTlhMjljNmUzYzBmZTc0ODhhNzlkYTQ4NCZYLUFtei1TaWduZWRIZWFkZXJzPWhvc3QifQ.0YSbEVusUF_KtTcWMp-X2aWQRnoLsA2EG3FrXN_5IUQ) I honestly can't make it work the way you did: Would you mind sharing a reproducible example?

Seems to work, that's good news. Thank you for the insight! However, I have some doubts on the reasons that led me to use definitely non-nullable types, I'm a bit...

Sorry for the delay @mahramane, I totally forgot to answer you 😅 KMP will come to the next release, it's just a matter of time 😉

I'm working on this project when I can dedicate time to it, and currently this is not possible because I have other things to manage.