search

tripalocal / website_v1.0

Tripalocal_V1
0 stars 0 forks source link

DO NOT remove the time field from the detail page #234

Closed yi-han closed 8 years ago

yi-han commented 8 years ago

For new product and itinerary, hide the time field instead of removing it.

kemblesong commented 8 years ago

It isn't removed, it's only rendered in the html if the experience is not a product. Is there anything wrong with this?

yi-han commented 8 years ago

Keep rendering the time in all cases, otherwise clicking "request" will cause an error, as the time is missing.

On 13 October 2015 at 10:37, Kemble Song notifications@github.com wrote:

It isn't removed, it's only rendered in the html if the experience is not a product. Is there anything wrong with this?

— Reply to this email directly or view it on GitHub https://github.com/tripalocal/website_v1.0/issues/234#issuecomment-147549321 .

kemblesong commented 8 years ago

Ok, should be fixed in commit af56d7b on test-server I made a pull request

yi-han commented 8 years ago

The problem still exists

yi-han commented 8 years ago

Display the time in all cases When uploading the experience, set the blockout time, so it only shows one time slot per day

yi-han commented 8 years ago

Fixed