Skip to content

Il centroide del comune non arrivava alla risposta - #145

Merged
gzileni merged 1 commit into
mainfrom
fix-comune-coords
Sep 30, 2026
Merged

gzileni merged 1 commit into
mainfrom
fix-comune-coords

Conversation

@gzileni

@gzileni gzileni commented Sep 30, 2026

Copy link
Copy Markdown
Contributor

Il repo lo produceva, il DTO non lo dichiarava, e response_model lo toglieva in silenzio: la colonna riceveva righe senza lon/lat e cliccare un comune non muoveva la mappa — cioè esattamente la cosa che #144 doveva sistemare.

Trovato provando la UI dopo il merge, non dai test: nessuno guardava la risposta, solo il repo. Ora lo fa uno.

🤖 Generated with Claude Code

Il repo lo produceva, il DTO non lo dichiarava, e `response_model` lo
toglieva in silenzio: la colonna riceveva righe senza coordinate e cliccare
un comune non muoveva la mappa — cioè esattamente la cosa che la modifica
doveva sistemare.

Un campo che il repo produce e il DTO non dichiara sparisce senza un errore
da nessuna parte, quindi ora lo verifica un test sulla risposta e non sul
repo.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@gzileni
gzileni merged commit 1146e28 into main Sep 30, 2026
6 checks passed
@gzileni
gzileni deleted the fix-comune-coords branch September 30, 2026 09:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant