Adam1302 / SpringBoot-AML

A SpringBoot Project dedicated to learning about works of Art, Media, and Literature
0 stars 2 forks source link

FEATURE: Add @JsonProperty to DTO Fields #56

Closed kamaljitkaur98 closed 5 months ago

kamaljitkaur98 commented 5 months ago

Ticket

Closes #21

Information

What? Add @JsonProperty to DTO Fields after DTO creation

Why? The coupling of JSON and model fields is unnecessary and makes us use snake_case. Updated for BookDTO (dto) and Book (model)

How? https://www.baeldung.com/jackson-annotations#1-jsonproperty

Code Checklist

sonarcloud[bot] commented 5 months ago

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

kamaljitkaur98 commented 5 months ago

@Adam1302 please do test once before merge. I did test with one dummy record on my side

Adam1302 commented 5 months ago

Everything look good to me so I'm going to merge @kamaljitkaur98 . If there are other changes we can include them in another PR.