popcodeorg / popcodeorg/popcode
Refactor instructions editor to follow controlled component pattern
Personne n'a encore pris cette issue.
- Langage dominant
- JavaScript
- Étoiles
- 191
- Forks
- 143
- Merge moyen
- 4 j 11 h
- PR mergées (30 j)
- 5
Description
Currently the instructions editor is an uncontrolled component, which is to say that intermediate in-progress editing state is stored only in the DOM.
React officially recommends using controlled components instead, which stores intermediate state in React (in our case Redux) state, and has the form value mirror the props given. This is probably the right approach here (but maybe it’s not! Further research may be called for).
Probably best not to sweat this too much until #1394 is done (or at least concurrently with that task).
Guide de contribution
Aucun guide de contribution indexé pour ce dépôt
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Piste de recherche
Commencez par examiner l’éditeur d’instructions et la documentation React liée sur les composants contrôlés, puis vérifiez comment #1394 affecte le travail. Le refactoring est terminé lorsque l’état intermédiaire de l’édition est stocké dans l’état de React ou de Redux et que la valeur du formulaire reflète ses props.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- javascript, react, redux
- Domaine
- frontend
- Type d'issue
- Refactorisation
- Difficulté
- 5/5
- Temps estimé
- Plus d'une semaine
- Activité
- À l'abandon
- Clarté
- À clarifier
- Accessibilité débutants
- 25/100