search

italia / docs-italia-comandi-conversione

Comandi di conversione di formato per Docs Italia
GNU Affero General Public License v3.0
7 stars 3 forks source link

errore nell'uso delle opzioni json #59

Closed danse closed 6 years ago

danse commented 6 years ago 
 docs-italia-esempiformattazione-docs (master)*$ converti esempio.docx --opzioni-json opzioni-converti.json 
Invalid option `--opzioni-json'

Usage: converti ([documento.ext] [--collegamento-normattiva] [--celle-complesse]
                [--preserva-citazioni] [--dividi-sezioni] | [--opzioni-json ARG]
                | [--version])

chiedo scusa @yakky, avrei dovuto testare meglio. Sei già incappato in questo errore?

danse commented 6 years ago

risolvere questo mi porta a ristrutturare le opzioni. attualmente il documento su cui converti agisce è opzionale, con un valore di default documento.docx. credo che non abbia senso, lo renderò un argomento obbligatorio

danse commented 6 years ago

attualmente è possibile specificare il documento tramite un campo documento nel JSON, ma non penso serva a granché ... credo sia meglio lanciare converti documento.ext --opzioni-json opzioni.json

danse commented 6 years ago

:thinking: più che un bug è in realtà una scelta di interfaccia che mi sembra molto infelice, ma aspetto l'opinione di @yakky

danse commented 6 years ago

ovviamente possiamo anche continuare a supportare documento nel JSON, ed aggiungere la possibilità di indicare il documento come argomento ... gli utenti però in quel caso vorrebbero sapere quale valore ha il sopravvento. mi sembra una complicazione che non serve a granchè

yakky commented 6 years ago

si, concordo. ha senso rendere obbligatorio il path del documento e togliere l'attributo documento dal json, mi sembra più lineare

danse commented 6 years ago

hey, il problema è risolto nell'ultima release, così non siamo bloccati nel convertitore web! https://github.com/italia/docs-italia-comandi-conversione/releases/tag/v0.5 :)