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

Ordine delle opzioni irrilevante in converti #69

Open danse opened 6 years ago

danse commented 6 years ago

o per lo meno un messaggio d'errore migliore perchè quello che abbiamo ora è fuorviante

 docs-italia-esempiformattazione-docs (master)*$ converti --opzioni-json opzioni-converti.json esempio-odt.odt
 docs-italia-esempiformattazione-docs (master)*$ converti  esempio-odt.odt --opzioni-json opzioni-converti.json
Invalid option `--opzioni-json'

Usage: converti (documento.ext [--collegamento-normattiva] [--celle-complesse]
                [--preserva-citazioni] [--dividi-sezioni] | documento.ext
                --opzioni-json ARG | --version)
  converte il documento in formato rST.
  Nel caso più semplice, potete convertire un documento lanciando:

      $ converti documento.ext

  La documentazione più dettagliata si trova al seguente indirizzo:
  https://github.com/italia/docs-italia-comandi-conversione/blob/master/doc/comandi/converti.md
 docs-italia-esempiformattazione-docs (master)*$ 
danse commented 6 years ago

@yakky tieni presente che esiste questo problema, mi ha creato molta confusione oggi ...

danse commented 6 years ago

anche l'help text prodotto dalla libreria che uso è fuorviante perché mostra le opzioni nell'ordine che causa l'errore ... :unamused:

yakky commented 6 years ago

Ovviamente quello sbagliato è anche l'ordine usato nel convertitore web :) Faccio una patch

danse commented 6 years ago

mentre mettiamo su questa logica è un buon momento per testare il report degli errori ... arrivano notifiche da Sentry o via mail? sono comprensibili? Il mio obiettivo sarebbero notifiche tali che tu possa fare copia ed incolla su una nuova issue dei comandi più o meno come ho fatto io sopra

danse commented 6 years ago

allegando il doc, nel caso generale

danse commented 6 years ago

non credo abbia senso investire nell'interfaccia dei comandi in questa fase del progetto visto che l'interfaccia principale che proporremo agli utenti è quella web. inoltre aggiornare l'interfaccia dei comandi richiede aggiornamenti al convertitore web, che invece preferiamo tenere stabile per concentrarci su attività che portino un beneficio più tangibile. tolgo questa issue dalla milestone

yakky commented 6 years ago

@danse si, vengono loggati su sentry, con le opzioni e il link per scaricare il file