[!WARNING]
This work was not approved for the on-season (as of 06/14/24) and should not be merged in without the SO verifying they want it. It should remain saved until the off-season and made part of the normal bundle of work.
[!CAUTION]
This work was off of the existing production branch that had its targets pointing to dev. This may require the branch being updated to point point at .dev again if this has been changed to .prod
Pull Request Standards
[x] The title of the PR is accurate
[x] The title includes the type of change [HOTFIX, FEATURE, etc]
[x] The PR title includes the ticket number in format of [INSPECT-###]
[x] Documentation is updated to reflect change
Description
This PR includes the following proposed change(s):
Add new boolean flag to Inspections formDidValidate
boolean only triggers when user exits inspection, and validation passed by conventional method
if user bypasses validation, the boolean remains false :)
When user goes to submit a shift, the shift checks that all inspections have formDidValidate == true
If this fails, it marks all failing inspections with new status enum .Errors
.Errors enum displays red on screen
User is alerted their inspection fails due to unvalidated inspections
Submission is cancelled, no data is sent, no duplicate entries made
Converted several string == "" checks to the swift built in flag string.isEmpty
Pull Request Standards
HOTFIX
,FEATURE
,etc
][INSPECT-###]
Description
This PR includes the following proposed change(s):
formDidValidate
all
inspections haveformDidValidate == true
.Errors
string == ""
checks to the swift built in flagstring.isEmpty