-
Notifications
You must be signed in to change notification settings - Fork 8
Description
In the process of checking our external engagements to see if the date filter wasn’t working (issue #44), I discovered that leaving any of the fields blank for # female students, # male students, etc. will prevent the external engagement from appearing on Table D (/reporting/external-engagement).
I’m not sure how external engagement counts were submitted with a couple of blank fields since all of the fields default to 0 and uploading a blank Excel spreadsheet populates the blank fields with 0. I guess this user manually deleted a 0 and left it blank? As of now there is no check to make sure none of the fields are blank. For now I just added a note to ask the participants to double-check all of the fields to make sure they’re not blank, but maybe we would want to add validation code for that.