plotly/falcon

Refactor Preview.react.js and Settings.react.js

Open

#362 aperta il 1 feb 2018

Vedi su GitHub
 (15 commenti) (1 reazione) (0 assegnatari)JavaScript (279 fork)batch import
discussiongood first issuehelp wantedrefactor

Metriche repository

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

Descrizione

cc/ @kcolton @chriddyp @tarzzz

I'm starting this discussion prompted by issues:

  • #360 (poor performance of code editor)
  • #361 (duplicated sql-schemas requests)

Although the code editor lives in CodeEditorField.react.js and the schema preview in TableTree.react.js, it's very likely that we'll have to touch code in their parent components Preview.react.js and Settings.react.js.

I'd like to use this issue to discuss any significant changes to Preview.react.js and Settings.react.js before creating a PR.

Guida contributor

Direzione di ricerca
Analizza Preview.react.js e Settings.react.js, traccia il flusso di dati verso i componenti figli come CodeEditorField e TableTree. Rivedi gli issue #360 e #361 per comprendere i problemi di prestazioni e le richieste duplicate. Investiga la gestione dello stato e il passaggio di props per identificare opportunità di refactoring per migliorare le prestazioni e ridurre la ridondanza.
Tech stack
javascriptreact
Dominio
frontenddesktop
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-2 giorni
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
ReactJavaScriptGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60