validatorjs/validator.js

Feature Request: Currency: Option to Require Thousands Separator

Open

#912 geöffnet am 15. Okt. 2018

Auf GitHub ansehen
 (7 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (22.497 Stars) (2.253 Forks)batch import
good first issuehelp neededpr-raised🙋‍♂️feature-request

Beschreibung

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

Contributor Guide

Tech Stack
javascript
Domain
backend
Issue Type
feature
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 hours
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
stale
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
JavaScriptregex
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70
Research-Richtung
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.