search

AnalogIO / analog-core

.NET 8 backend for Cafe Analog's coffee card app
https://core.prd.analogio.dk/swagger
MIT License
4 stars 1 forks source link

Changes Users to property #267

Closed A-Guldborg closed 6 months ago

A-Guldborg commented 6 months ago

With the new Json serializer from #266, fields are no longer serialized.

Preferably, we would have added the Json options options.JsonSerializerOptions.IncludeFields = true; to ensure that no other functionality broke. However, due to an issue raised here, the schema in Swagger will not show fields despite setting this option. Thus, our API-generation would not work for e.g. Shifty that uses this.

Instead, I have looked through CoffeeCard.Models and not found other examples where we are using fields and not properties.

sonarcloud[bot] commented 6 months ago

Quality Gate Passed Quality Gate passed

Issues
1 New issue
0 Accepted issues

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

See analysis details on SonarCloud