Skip to content
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

Supporting sqlx Json<T> #952

Open
ElysaSrc opened this issue May 27, 2024 · 3 comments
Open

Supporting sqlx Json<T> #952

ElysaSrc opened this issue May 27, 2024 · 3 comments
Labels
enhancement New feature or request investigate Futher investigation needed before other action

Comments

@ElysaSrc
Copy link

I think it'd be nice to support (behind a feature flag) the Json<T> because, currently the typing is lost on this element on the OpenAPI generated file.

@juhaku
Copy link
Owner

juhaku commented Sep 26, 2024

@ElysaSrc Could you elaborate bit more how you would like the support to be? How it should behave?

@ElysaSrc
Copy link
Author

ElysaSrc commented Sep 26, 2024

@ElysaSrc Could you elaborate bit more how you would like the support to be? How it should behave?

Currently I must annotate all Json fields of my struct that is requested though sqlx with the inner type, I was wondering if maybe it could be inferred so when we have a json it directly infer a T type in the openapi scheme ?

@juhaku
Copy link
Owner

juhaku commented Oct 1, 2024

Okay, so it should infer the inner type. This needs some looking into to assess what can be done there.

@juhaku juhaku added enhancement New feature or request investigate Futher investigation needed before other action labels Oct 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request investigate Futher investigation needed before other action
Projects
Status: No status
Development

No branches or pull requests

2 participants