Closes #177. This issue was because unconfirmed special event volunteers were not being included in the "Drivers Only", "Packers Only", or "Driving & Packing" counts. For instance, the "Drivers Only" total was being calculated by summing the "Only Driver Count Including Unconfirmed" property for the general event, with the "Only Driver Count" property (which only counts confirmed drivers) of each special event.
I fixed this by updating the properties of the special events that were being summed to be the properties that included unconfirmed volunteers: I replaced "Only Driver Count" with "Only Driver Count Including Unconfirmed", "Only Distributor Count" with "Only Distributor Count Including Unconfirmed", and "Driver and Distributor Count" with "Driver and Distributor Count Including Unconfirmed".
Screenshots (for UI changes - otherwise delete this section)
Before this PR:
After this PR:
Checklist
[x] Added link to existing Issue (created issue if there wasn't one already)
[x] Added screenshots before/after for UI changes
[x] PR is from branch pushed to this repo so that it will trigger Railway PR deployment
Description of this change
Closes #177. This issue was because unconfirmed special event volunteers were not being included in the "Drivers Only", "Packers Only", or "Driving & Packing" counts. For instance, the "Drivers Only" total was being calculated by summing the "Only Driver Count Including Unconfirmed" property for the general event, with the "Only Driver Count" property (which only counts confirmed drivers) of each special event.
I fixed this by updating the properties of the special events that were being summed to be the properties that included unconfirmed volunteers: I replaced "Only Driver Count" with "Only Driver Count Including Unconfirmed", "Only Distributor Count" with "Only Distributor Count Including Unconfirmed", and "Driver and Distributor Count" with "Driver and Distributor Count Including Unconfirmed".
Screenshots (for UI changes - otherwise delete this section)
Before this PR:
After this PR:
Checklist