angular-ui/ui-grid

Rewrite notifyDataChange to be a grid eventing process instead

Open

#3.603 aberto em 27 de mai. de 2015

Ver no GitHub
 (0 comments) (0 reactions) (1 assignee)JavaScript (2.496 forks)batch import
good first issuetype: chore

Métricas do repositório

Stars
 (5.395 stars)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

Description

Thinking about this, there's no reason this couldn't use grid.raise and grid.on instead. And that'd be much more in pattern, avoiding having a special thing that confuses people.

It shouldn't be overly hard, it's largely a pattern-based search and replace thing.

Guia do colaborador

Direção de pesquisa
Entenda como grid.raise e grid.on são usados em outros lugares na base de código. Encontre todos os usos de notifyDataChange e substitua os por chamadas de evento de grade equivalentes.
Pilha de tecnologia
javascript
Domain
frontend
Tipo Issue
Refatoração
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.
Principalmente claro
Prerequisites
AngularJSui grid eventing
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70