validatorjs/validator.js

Feature Request: Currency: Option to Require Thousands Separator

Open

#912 aberto em 15 de out. de 2018

Ver no GitHub
 (7 comments) (0 reactions) (0 assignees)JavaScript (22.497 stars) (2.253 forks)batch import
good first issuehelp neededpr-raised🙋‍♂️feature-request

Description

We're using this to help validate data that includes currency. We want to make sure that a comma separator is always present.

Guia do colaborador

Pilha de tecnologia
javascript
Domain
backend
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.
clear
Prerequisites
JavaScriptregex
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70
Direção de pesquisa
Examine the existing `isCurrency` function in validator.js (likely in `src/lib/isCurrency.js`). The request is to add an option to require a thousands separator (comma). Look at the current implementation and its options. Discuss in the issue comments (7 comments) for any proposed solutions. Modify the function to accept a `require thousands separator` boolean and update tests accordingly.