AnSavvides/d3act

Add better error handling

Open

#3 ouverte le 17 août 2015

Voir sur GitHub
 (0 commentaires) (0 réactions) (0 assignés)JavaScript (300 stars) (26 forks)batch import
enhancementhelp wanted

Description

Right now, it's all a little bit of magic; if you don't pass everything that's required to the Chart component, you won't know what's gone wrong. We should raise some informative errors to make it easier to figure out what's going wrong.

Guide contributeur

Stack technique
javascriptreact
Domaine
frontend
Type d'issue
feature
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.
1-3 hours
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 knowledgeFamiliarity with d3act's Chart component
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
80
Direction de recherche
The issue requests better error handling in the Chart component. Look at the current implementation of Chart in the d3act source code (likely in a file like `src/Chart.js` or similar). Identify which props are required and currently unchecked. Add validation at the start of the component to check for missing required props and throw informative error messages. Consider using prop types or custom validation. The fix is localized to this one component.