-
Notifications
You must be signed in to change notification settings - Fork 40.7k
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
Auto-configure Hibernate with a JsonFormatMapper #42676
base: main
Are you sure you want to change the base?
Conversation
Thanks, @nosan. Before merging this, we should consider if we want a way to disable it (an application property?) as it has the potential to be a breaking change. |
Thanks, @wilkinsona |
I also used |
Thanks, @nosan. While looking for some inspiration for a property name, a couple of other things came to mind. I've noted them in a comment on 33870. I'll put this on hold for now until we've had a chance to figure those out. |
Thanks, @wilkinsona. That makes sense. What do you think about the following property format?
By default, for example, |
I'm not sure, unfortunately. I'm wondering if we need to widen the scope of the existing MVC-specific property and rename it to be more generally applicable. Let's keep that discussion to #33870 and then come back here once we're ready to update the proposal. |
#33870