Salvataggio bozza nuovo argomento

[Rhidian (Rhidian Bramley)]

Priorità: Bassa

Piattaforma: Discourse mobile iOS (non testato su altre piattaforme). Riprodotto su Discourse meta e try.Discourse.org

Descrizione: Salvare una bozza di Nuovo Argomento quando richiesto. Il pulsante Apri Bozza non funziona.

Passaggi per riprodurre:

1. Vai alla home page di Discourse.
2. Clicca su + Nuovo Argomento.
3. Inserisci nell’oggetto ‘Testing Drafts’. Inserisci nel corpo del compositore ‘Testing’. Lascia senza categoria.
4. Clicca Annulla (icona cestino su Discourse mobile). Ti viene chiesto se vuoi salvare una bozza per dopo: seleziona quell’opzione.
5. Il pulsante + Nuovo Argomento cambia in + Apri Bozza.
6. Clicca su + Apri Bozza. Non succede nulla.
7. Controlla la cartella Bozze: nessuna bozza è stata salvata.

Conseguenza: Questo mi ha temporaneamente impedito di creare un nuovo Argomento.

Soluzione alternativa: Cliccando ripetutamente sul pulsante inattivo + Apri Bozza, l’app mobile di Discourse si blocca infine e ritorna alla selezione del dominio. Torna su Discourse: il pulsante + Apri Bozza è stato resettato ed è tornato a + Nuovo Argomento.

[jean.perez (Jean)]

Il titolo non è abbastanza lungo per superare questa convalida

https://github.com/discourse/discourse/blob/master/app/assets/javascripts/discourse/app/models/composer.js#L1184

Ma mi chiedo, perché il codice risolve invece di rifiutare e poi catturare gli errori?

Penso che gli errori possano essere catturati qui

https://github.com/discourse/discourse/blob/master/app/assets/javascripts/discourse/app/controllers/composer.js#L1244

qualcosa del genere

this._saveDraftPromise = model.saveDraft().then(() => {
    this._lastDraftSaved = Date.now();
    ...
}, error => {
    // gestione dell'errore qui
}).finally(() => {
    this._saveDraftPromise = null;
});

@eviltrout hai qualche opinione?


[eviltrout (Robin Ward)]

Questo codice di salvataggio è purtroppo tra i più vecchi e spesso i più fragili. Nel corso degli anni abbiamo aggiunto molte funzionalità, e alcune cose, come il salvataggio delle bozze che può avere esito positivo o negativo, richiedono funzionalità personalizzate che lo complicano ulteriormente.

Penso che la tua intuizione sia probabilmente corretta, ma è impossibile per me confermarlo senza vedere come vengono eseguiti tutti i nostri test. Se in futuro avrai tempo per una PR, sarebbe fantastico.

[SonsOfAhriman (Ryan Huddleston)]

Sono riuscito a riprodurre questo bug su MacOS Chrome in Discourse meta seguendo i passaggi indicati da Rhidian di seguito:

@Rhidian

Passaggi per la riproduzione:

1. Vai alla home page di Discourse.
2. Clicca su + Nuovo argomento.
3. Digita nell’oggetto ‘Testing Drafts’. Digita nel corpo del compositore ‘Testing’. Lascia senza categoria.
4. Clicca su Annulla (icona del cestino su Discourse mobile). Ti viene chiesto se vuoi salvare una bozza per dopo: seleziona quell’opzione.
5. Il pulsante + Nuovo argomento cambia in + Apri bozza.
6. Clicca su + Apri bozza. Non succede nulla.
7. Controlla la cartella Bozze: non è stata salvata alcuna bozza.

[nbianca (Bianca)]

Ciao,

Grazie per il rapporto sul bug. È stato risolto nell’ultima versione di Discourse. Il problema era che proponeva di salvare bozze molto brevi anche quando non dovrebbe farlo.