angular-ui/ui-grid

Rewrite notifyDataChange to be a grid eventing process instead

Open

#3603 aperta il 27 mag 2015

Vedi su GitHub
 (0 commenti) (0 reazioni) (1 assegnatario)JavaScript (2496 fork)batch import
good first issuetype: chore

Metriche repository

Star
 (5395 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

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.

Guida contributor

Direzione di ricerca
Comprendi come grid.raise e grid.on vengono utilizzati altrove nel codebase. Trova tutti gli usi di notifyDataChange e sostituiscili con chiamate di eventi grid equivalenti.
Tech stack
javascript
Dominio
frontend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
AngularJSui grid eventing
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70