validatorjs/validator.js

Feature Request: Currency: Option to Require Thousands Separator

Open

#912 ouverte le 15 oct. 2018

Voir sur GitHub
 (7 commentaires) (0 réactions) (0 assignés)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.

Guide contributeur

Stack technique
javascript
Domaine
backend
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.
clear
Prérequis
JavaScriptregex
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70
Direction de recherche
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.