Merge pull request #308 from Ohtuilmo/fix/sprint-zero-timelogs

Fix bug with sprint number validation
Ohtuilmo · May 14, 2024 · c699f9e · c699f9e
2 parents 1f059d8 + 079f633
commit c699f9e
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/backend/controllers/timeLogs.js b/backend/controllers/timeLogs.js
@@ -4,7 +4,7 @@ const db = require('../models/index')
 const { Sequelize } = require('sequelize')
 
 const validateTimeEntry = ({ sprint, date, minutes, description }) => {
-  if (!sprint || !date || isNaN(parseFloat(minutes)) || !description) {
+  if (isNaN(sprint) || !date || isNaN(parseFloat(minutes)) || !description) {
     return 'All fields must be filled.'
   }
   if (parseFloat(minutes) <= 0) {