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

Accommodation: Allow single arrival/departure date #4164

Merged
merged 1 commit into from Nov 20, 2019

Conversation

@bpedersen2
Copy link
Contributor

bpedersen2 commented Nov 19, 2019

Allow to restrict the date range to a single day for
arrival/departue in the setup form.

@ThiefMaster

This comment has been minimized.

Copy link
Member

ThiefMaster commented Nov 19, 2019

Does it make sense to select accommodation at all in this case? In case payment is used, the price would be zero (0 nights), so usually people are supposed to select "no accommodation" in such a case.

@bpedersen2

This comment has been minimized.

Copy link
Contributor Author

bpedersen2 commented Nov 20, 2019

Does it make sense to select accommodation at all in this case? In case payment is used, the price would be zero (0 nights), so usually people are supposed to select "no accommodation" in such a case.

There you misunderstand the change, it is about allowing only a single arrival and/or departure date to be selected. Currently you always need to allow at least 2 days for arrival and departure.

Allow to restrict the date range to a single day for
arrival/departue in the setup form.
@ThiefMaster ThiefMaster force-pushed the bpedersen2:dates branch from 6557ba3 to fbec3f9 Nov 20, 2019
@ThiefMaster ThiefMaster merged commit fbec3f9 into indico:2.2-maintenance Nov 20, 2019
1 check was pending
1 check was pending
continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
@ThiefMaster

This comment has been minimized.

Copy link
Member

ThiefMaster commented Nov 20, 2019

Make sense. Thanks!

@bpedersen2 bpedersen2 deleted the bpedersen2:dates branch Nov 21, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.