Skip to content
This repository has been archived by the owner on Jul 24, 2020. It is now read-only.

Missed reservations - what do with? #594

Closed
2 tasks done
dgoerger opened this issue Jul 1, 2014 · 6 comments · Fixed by #1051
Closed
2 tasks done

Missed reservations - what do with? #594

dgoerger opened this issue Jul 1, 2014 · 6 comments · Fixed by #1051

Comments

@dgoerger
Copy link
Contributor

dgoerger commented Jul 1, 2014

Chris pointed out at our last meeting that we currently have a rake task deleting missed reservations at 5am every day, but we still have stats for missed on the user pages and in the Reservations list. This is silly. We can

  1. not delete missed reservations and make sure our code is intelligent enough to handle that for double-counting and validations purposes (we should do the latter regardless). downside: list will become very very long with the age of the database.
  2. delete missed reservations after some period of time, e.g. a week.
  3. continue deleting missed reservations and remove the views on the user profile pages and reservations lists which do not make any sense.

  • Add setting in app config for missed reservation deletion frequency (see Hide old blackouts #242 for similar behavior with old blackouts)
  • remove missed reservation stats from user profile page
@squidgetx
Copy link
Contributor

I vote 2

@dgoerger
Copy link
Contributor Author

dgoerger commented Jul 1, 2014

for option 2, we'd just need to make sure to make this behavior clear in the views. the user profile pages stats for missed reservations is probably then also useless and should be removed.

@mnquintana
Copy link
Contributor

I think 2 is the best option.

@orenyk
Copy link
Contributor

orenyk commented Aug 26, 2014

Looks like this was never implemented but should be pretty straightforward. I'm moving it to v4.1 and adding a checklist to the issue description.

@orenyk orenyk added this to the 4.1 milestone Aug 26, 2014
@orenyk orenyk removed the 1 - Ready label Sep 3, 2014
@orenyk orenyk modified the milestones: 4.0.1, 4.1 Sep 21, 2014
@orenyk orenyk modified the milestones: 4.1.0, 4.0.1 Oct 27, 2014
@orenyk orenyk added 1 - Ready and removed 1 - Ready labels Nov 3, 2014
@orenyk
Copy link
Contributor

orenyk commented Nov 26, 2014

Looks like the blackouts_exp_time parameter was missing from strong params... whoops!

@orenyk
Copy link
Contributor

orenyk commented Nov 26, 2014

woo testing rake tasks is fun (see here). Once I get them working I'll push and issue a PR.

@orenyk orenyk self-assigned this Nov 26, 2014
orenyk added a commit that referenced this issue Nov 30, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants