yogthos/memory-hole

event handlers are not pure

Aberta

#22 aberto em 29 de out. de 2016

Ver no GitHub
 (1 comentário) (0 reação) (0 responsável)Clojure (25 forks)github user discovery
enhancementhelp wanted

Métricas do repositório

Stars
 (264 estrelas)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

Event handlers in re-frame should be pure. In the handlers there are side effects created by calling ajax functions.

https://github.com/yogthos/memory-hole/blob/master/src/cljs/memory_hole/handlers/admin.cljs#L15

The re-frame / pure way would be to write an effect handler which handles the execution of the state change. Have a look at the following library to see how this can be solved in a pure way:

https://github.com/Day8/re-frame-http-fx

Guia do colaborador

Direção de pesquisa
Revise a biblioteca re frame http fx e refatore os manipuladores de admin para usar manipuladores de efeito para chamadas ajax.
Pilha de tecnologia
clojure
Domain
frontend
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
Clojurere frame
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60