Import Excel tollerante (fase 9) - #51
Merged
Merged
Conversation
Il consulente userà file "simili" al suo modello, non identici: prima bastava un titolo sopra le intestazioni, un foglio in più davanti o "Codice" al posto di "Codice Conto Mastro" per fermare l'import. - la riga di intestazione si cerca nelle prime 20 righe e il foglio fra tutti quelli del file; se i fogli buoni sono più d'uno si sceglie - sinonimi per colonne, sezioni e sotto-classificazioni; numerazioni, punteggiatura, maiuscole e "v/" non contano - una sezione sconosciuta si abbina a mano dalla schermata: l'anteprima si rifà e l'import scrive con lo stesso abbinamento, annotato nel documento di origine - un conto senza TIPO prende quello della sezione; un TIPO incoerente con la sezione si ferma nel riepilogo invece di entrare col segno sbagliato (stessa regola dei conti scritti nel programma: typesForSection ora vive nel parser e il ledger la riusa) - importi come testo: "1.234" migliaia, "(1.234,56)" e "1.234,56-" negativi, "-" vuoto; una cella in formato % (0,7) diventa 70 Provato anche sull'app compilata, via API: file "da un altro studio" → anteprima → abbinamento → import → conti con sezione, tipo, 70% e sotto-classificazione giusti. I due test sul file vero del consulente restano verdi. 170 test. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Perché
Il consulente userà file Excel simili al suo modello, non identici (AGENTS.md §11.1). Prima bastava un titolo sopra le intestazioni, un foglio in più davanti o "Codice" al posto di "Codice Conto Mastro" per fermare l'import.
Cosa cambia
Resta valida la regola di sempre: niente si scrive senza anteprima e conferma.
Verifiche
Closes #47 per la parte di codice; la prova su file veri di più studi resta aperta nella fase 9.
🤖 Generated with Claude Code