idurar/idurar-erp-crm

Able to add fields in invoices without item name

Open

#207 geöffnet am 8. Juli 2023

Auf GitHub ansehen
 (7 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (4.308 Stars) (1.036 Forks)batch import
good first issue🎨 Frontend🐛 bug

Beschreibung

image

Hey I dont think users should be able to add fields in the invoices without an item name. I just entered spaces on the item name and item name field isn't treating it as empty string. Also I am able to enter quantities in points like 0.1, 0.3 and even 0. I think that too shouldn't be allowed because quantities in points or 0 doesn't make sense. This is a minor bug which needs to be fixed. Can you assign this to me so I can fix this bug?

Contributor Guide

Tech Stack
javascriptreactnodejsexpressmongodb
Domain
frontendbackend
Issue Type
bug
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
under 1 hour
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
ReactNode.js
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
80
Research-Richtung
The issue requires adding validation to prevent empty item names (only spaces) and non positive integer quantities in invoice items. Examine the invoice creation form component (likely in React) for frontend validation. Also check the backend API endpoint for invoice creation (likely Node.js/Express) to add server side validation for the item name and quantity fields. The item name should be trimmed and checked for emptiness, and quantity should be an integer greater than 0. Look at existing validation patterns in the codebase for consistency.