christopherdro/react-native-calendar

Inconsistency in passing value to callbacks

Open

#54 aberto em 19 de jul. de 2016

Ver no GitHub
 (1 comment) (0 reactions) (0 assignees)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.

Guia do colaborador

Pilha de tecnologia
react native
Domain
mobile
Tipo Issue
feature
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 hours
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
stale
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
mostly clear
Prerequisites
familiarity with react native calendar callbacksknowledge of moment.js
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
30
Direção de pesquisa
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.