frintjs/frint

frint-react: Move subscription to componentDidMount in `observe` HoC

Open

#417 ouverte le 11 avr. 2018

Voir sur GitHub
 (1 commentaire) (1 réaction) (0 assignés)JavaScript (746 stars) (37 forks)batch import
help wanted

Description

At the moment, it's happening in componentWillMount, which is not recommended.

Guide contributeur

Stack technique
javascriptreact
Domaine
frontend
Type d'issue
refactor
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
under 1 hour
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
stale
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
basic React lifecycle knowledge
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
90
Direction de recherche
The issue asks to move the subscription from componentWillMount to componentDidMount in the `observe` higher order component. Since componentWillMount is deprecated in newer React versions, this is a straightforward migration. Look for the relevant HoC file in the frint react package, find where componentWillMount is used, and move the subscription logic to componentDidMount. Ensure to update any related tests if they exist.