christopherdro/react-native-calendar

Inconsistency in passing value to callbacks

Open

#54 ouverte le 19 juil. 2016

Voir sur GitHub
 (1 commentaire) (0 réactions) (0 assignés)JavaScript (544 stars) (250 forks)batch import
enhancementhelp wanted

Description

I spotted inconsistency in callback values. Namely these: onSwipePrep and onSwipeNext receive moment instance. onDateSelected receives date in plain string format.

Unifying this issue is essentially a breaking change, but it would be nice to be consistent.

Guide contributeur

Stack technique
react native
Domaine
mobile
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.
3
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.
mostly clear
Prérequis
familiarity with react native calendar callbacksknowledge of moment.js
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
30
Direction de recherche
Investigate the callback implementations in the source code to confirm which callbacks return moment instances and which return strings. The issue specifically mentions onSwipePrep, onSwipeNext, and onDateSelected. The fix would involve changing the callbacks to consistently return either moment objects or strings. Since this is a breaking change, consider adding a new prop or version. The repository is deprecated, so contribution may not be merged.