mikhael28/paretOS

Refactor: ContextPage component is very long and could benefit from some refactoring

Aperta

#156 aperta il 29 mar 2022

Vedi su GitHubGuida contributor
 (18 commenti) (0 reazioni) (1 assegnatario)JavaScript (45 fork)auto 404
Reactgood first issuerefactoringup for grabs

Metriche repository

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

Descrizione

Description

The component context/ContextPage.js clocks in at a whopping 569 lines of code! Looking through, a good amount of it appears to be repetitive code that could be refactored.

Goals

  • Reduce the total lines of code
  • Reduce the amount of repetitive code
  • Improve code efficiency and readability
  • Simplify state management for states that are always all updated at the same time
  • Add comments as necessary to help orient new contributors navigating the file for the first time.

Guida contributor

Direzione di ricerca
Suddividi ContextPage.js identificando pattern ripetuti, estrai hook o componenti riutilizzabili e semplifica la gestione dello stato. Usa gli obiettivi come lista di controllo.
Tech stack
javascriptreact
Dominio
frontend
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
JavaScriptReact
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
50