Salvando Rascunho Novo Tópico

[Rhidian (Rhidian Bramley)]

Prioridade: Baixa

Plataforma: Discourse mobile iOS (não testado em outras plataformas). Reproduzido no meta do Discourse e em try.Discourse.org

Descrição: Salvar um rascunho de Novo Tópico quando solicitado. O botão Abrir Rascunho não funciona.

Passos para reprodução:

1. Acesse a página inicial do Discourse.
2. Clique em + Novo Tópico.
3. Digite no campo de assunto ‘Testando Rascunhos’. Digite no corpo do compositor ‘Testando’. Deixe sem categoria.
4. Clique em Cancelar (ícone de lixeira no Discourse mobile). Você será perguntado se deseja Salvar um rascunho para mais tarde — selecione essa opção.
5. O botão + Novo Tópico muda para + Abrir Rascunho.
6. Clique em + Abrir Rascunho. Nada acontece.
7. Verifique a pasta Rascunhos — nenhum rascunho foi salvo.

Consequência: Isso me deixou temporariamente impossibilitado de criar um novo Tópico.

Solução alternativa: Clicar várias vezes no botão inativo + Abrir Rascunho eventualmente causa um travamento do site do Discourse no aplicativo móvel e retorna à seleção de domínio. Retorne ao Discourse — o botão + Abrir Rascunho foi limpo e voltou a ser + Novo Tópico.

[jean.perez (Jean)]

O título não é longo o suficiente para passar nesta validação

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

Mas eu me pergunto, por que o código está resolvendo em vez de rejeitar e, em seguida, capturar os erros?

Acho que os erros podem ser capturados aqui

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

algo assim

this._saveDraftPromise = model.saveDraft().then(() => {
    this._lastDraftSaved = Date.now();
    ...
}, error => {
    // tratamento de erro aqui
}).finally(() => {
    this._saveDraftPromise = null;
});

@eviltrout alguma opinião?


[eviltrout (Robin Ward)]

Esse código de salvamento é, infelizmente, um dos mais antigos e, muitas vezes, o mais frágil de todos. Ao longo dos anos, adicionamos muitas funcionalidades a ele, e algumas coisas, como salvar rascunhos, que podem ter sucesso ou falhar, precisam de funcionalidades personalizadas próprias, o que só o complica ainda mais.

Acho que sua intuição provavelmente está correta aqui, mas é impossível para mim confirmar sem ver como todos os nossos testes são executados. Se você tiver tempo para fazer um PR no futuro, seria ótimo.

[SonsOfAhriman (Ryan Huddleston)]

Consegui reproduzir esse bug no Chrome do MacOS no meta do Discourse, seguindo os passos de Rhidian abaixo:

@Rhidian

Passos para reprodução:

1. Acesse a página inicial do Discourse.
2. Clique em + Novo Tópico.
3. Digite no campo de assunto ‘Testando Rascunhos’. Digite no corpo do editor ‘Testando’. Deixe sem categoria.
4. Clique em Cancelar (ícone de lixeira no Discourse mobile). Você será perguntado se deseja salvar um rascunho para depois — clique nessa opção.
5. O botão + Novo Tópico muda para + Abrir Rascunho.
6. Clique em + Abrir Rascunho. Nada acontece.
7. Verifique a pasta Rascunhos — nenhum rascunho foi salvo.

[nbianca (Bianca)]

Olá,

Obrigado pelo relatório do bug. Ele foi corrigido na versão mais recente do Discourse. O problema era que o sistema oferecia salvar rascunhos muito curtos, mesmo quando isso não deveria acontecer.