-
Notifications
You must be signed in to change notification settings - Fork 0
Add more validation for "generate timesheet" #29
Copy link
Copy link
Open
Labels
enhancementNew feature or requestNew feature or request
Description
r.get("review") === data.review &&
r.get("notBillable") === data.notBillable &&
!r.get("verfiedBy") &&
!r.get("isDeleted")ig we can add the review and non billable checks but I don't think such an edge-case would ever occur.
same goes for already verified reports; if one is verified on one day, why would anyone create a second activity.
After all this seems to me like very bad client-side validation
Originally posted by @gianklug in #24 (comment)
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
enhancementNew feature or requestNew feature or request