Midburn/spark

Clean global scope in all controllers.

Aperta

#980 aperta il 29 nov 2018

Vedi su GitHubGuida contributor
 (1 commento) (0 reazioni) (1 assegnatario)JavaScript (44 fork)auto 404
Cleanupgood first issue

Metriche repository

Star
 (18 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

Overview of the problem

We have several functions declared globally in controller files. This can create issues hard to anticipate and solve.

Description

  1. We need to move all functions into scope. image image
  2. We need to make sure the functions aren't used in any other file (if so, we should create an AngularJS service to be shared between the controllers)
  3. Sanity after the cleanup - check most screens and buttons don't raise exceptions related to missing functions.

Expected behavior

Functions should be scoped

Actual behavior

Functions are declared outside of AngularJS's controllers call back

Guida contributor

Direzione di ricerca
Cerca le funzioni dichiarate al di fuori dei callback dei controller in tutti i file dei controller, verifica il loro utilizzo nell'intero codebase (ad esempio usando grep o la ricerca IDE), quindi spostale nell'ambito del controller o estraile in servizi AngularJS se condivise.
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.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
JavaScriptAngularJS
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70