First, instead of calculating past users in the controller, we add
previous_users method to the shift summary.
Second, the method ensures that we don't look into the future when trying to
find past users. Mostly this is done by filtering the users we count as
"other" users to users that show up for events only int he past. Obvious in
hindsight.
First, instead of calculating past users in the controller, we add
`previous_users` method to the shift summary.
Second, the method ensures that we don't look into the _future_ when trying to
find _past_ users. Mostly this is done by filtering the users we count as
"other" users to users that show up for events only int he past. Obvious in
hindsight.