Skip to content

Feature/wordpress integration - #3

Open
JacAbreu wants to merge 11 commits into
juanplopes:masterfrom
JacAbreu:feature/wordpress_integration
Open

Feature/wordpress integration#3
JacAbreu wants to merge 11 commits into
juanplopes:masterfrom
JacAbreu:feature/wordpress_integration

Conversation

@JacAbreu

Copy link
Copy Markdown

I developed a facility for automating posts on Wordpress CMS. I really hope that is helpful and I am available to hear any critica about what was implemented.

@juanplopes

Copy link
Copy Markdown
Owner

Vamos lá, listando o que me incomodou:

Problemas maiores (que me impedem de fazer o merge):

  • A desserialização de um estado de uma versão antiga faz com que a variavel ParticipantsDay fique nula, fazendo com que o DojoTimer dê erro ao tentar adicionar um novo participante (o que ocorre em toda rodada). Isso é BEM grave.
  • Especialmente, se a lista de participantes do dia estiver nula, o programa dá erro ao tentar postar no WordPress.
  • A funcionalidade de postar força o uso de http como protocolo, obrigando o usuário a enviar sua senha em plaintext over the wire. Pode até ter um padrão para não obrigar o usuário a especificar o protocolo, mas o padrão deveria ser https, pelo menos.

Problemas menores no código:

  • Diversos artefatos espalhados sem uso, como a interface IWordpress ou a classe vazia PostPublisher.
  • As classes que modelam as entidades do Wordpress deveriam estar separadas do resto dos helpers.
  • O nome dado ao formulário de publicação não faz muito sentido (PostResumeDojoForm).
  • Você deveria realmente insistir em usar a interface IWordpress. Essa classe herdando de XmlRpcClientProtocol e usando reflection para a invocação está muito ruim.

Problemas menores de interface:

  • Muitos campos para preencher e nada é salvo entre uma sessão e outra. Se você quer que as pessoas usem essa feature, deve torná-la fácil de usar. De preferência, todos os valores deveriam ser salvos (exceto a senha) entre uma sessão e outra, para facilitar publicar somente o que for diferente.
  • Por ter muitos campos, a fonte usada nos campos é minúscula (e não segue o padrão do resto do DojoTimer). Só quem estiver no computador na hora vai conseguir ler. Quem estiver na plateia não vai.
  • Diagramação da tela está terrível, com os botões jogados pela tela, labels desalinhados, etc.
  • Mistura de português com inglês nos labels/mensagens de erro
  • "Publish in dojorio.org", sendo que não necessariamente o post vai para lá
  • O botão de publicar (na barra de título) está antes do botão de configurações, o que é ruim do ponto de vista de UX.
  • O form de publicação está iniciando em qualquer lugar da tela. O padrão da aplicação é iniciar em CenterParent.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants