plotly/falcon

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

Ouverte

#362 ouverte le 1 févr. 2018

Voir sur GitHubGuide contributeur
 (15 commentaires) (1 réaction) (0 personne assignée)JavaScript (279 forks)batch import
discussiongood first issuehelp wantedrefactor

Métriques du dépôt

Stars
 (5 093 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

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.

Guide contributeur

Direction de recherche
Analysez Preview.react.js et Settings.react.js, tracez le flux de données vers les composants enfants tels que CodeEditorField et TableTree. Consultez les issues #360 et #361 pour comprendre les problèmes de performance et les requêtes en double. Examinez la gestion d'état et le passage de props pour identifier les opportunités de refactoring afin d'améliorer les performances et de réduire la redondance.
Stack technique
javascriptreact
Domaine
frontenddesktop
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-2 jours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
ReactJavaScriptGit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60