Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Show childcare requests even if childcare is disabled #509

Merged
merged 1 commit into from
Nov 23, 2016

Conversation

aellispierce
Copy link
Contributor

The problem:

Oftentimes we want to provide childcare for an event but disable it
shortly before the event so that we can provide an accurate count to
the babysitter and don't need to worry about last minute signups.
Unfortunately this removes the previous childcare signup information
from the organizer console, so that once childcare is disabled we no
longer have access to the information for those who have already
requested childcare.

The solution:

This makes it so that childcare signup information is shown even if childcare
is disabled for the event. A message is shown to let the user know if
childcare is disabled.

Now, if you disable childcare but have previous signups you'll see this:
image

If childcare is disabled and there were no previous signups you'll see this:
image

The problem:

Oftentimes we want to provide childcare for an event but disable it
shortly before the event so that we can provide an accurate count to
the babysitter and don't need to worry about last minute signups.
Unfortunately this removes the previous childcare signup information
from the organizer console, so that once childcare is disabled we no
longer have access to the information for those who have already
requested childcare.

The solution:

This makes it so that childcare signup information is shown even if childcare
is disabled for the event. A message is shown to let the user know if
childcare is disabled.
@aellispierce
Copy link
Contributor Author

I didn't see any view specs currently so I did not add one for this. I would however be happy to go back and add one if you'd like a view spec for this page.

@tjgrathwell tjgrathwell merged commit 495e70b into railsbridge:master Nov 23, 2016
@tjgrathwell
Copy link
Member

👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants