plotly/falcon

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

Open

#362 geöffnet am 1. Feb. 2018

Auf GitHub ansehen
 (15 Kommentare) (1 Reaktion) (0 zugewiesene Personen)JavaScript (279 Forks)batch import
discussiongood first issuehelp wantedrefactor

Repository-Metriken

Stars
 (5.093 Stars)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

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.

Contributor Guide

Research-Richtung
Analysieren Sie Preview.react.js und Settings.react.js, verfolgen Sie den Datenfluss zu untergeordneten Komponenten wie CodeEditorField und TableTree. Überprüfen Sie die Issues #360 und #361, um Leistungsprobleme und doppelte Anfragen zu verstehen. Untersuchen Sie das Zustandsmanagement und die Props Übergabe, um Refactoring Möglichkeiten zur Verbesserung der Leistung und Reduzierung von Redundanzen zu identifizieren.
Tech Stack
javascriptreact
Domain
frontenddesktop
Issue Type
Refactoring
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-2 Tage
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
ReactJavaScriptGit
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60