Closes #608 - Lists event times as day + time

mozilla-mobile / prox

[INACTIVE] A search and discovery app for the "here & now". We're experimenting with ideas on mobile that can better surface content from the open web.

https://wiki.mozilla.org/New_Mobile_Experience

Mozilla Public License 2.0

22 stars 12 forks source link

Closes #608 - Lists event times as day + time #614

Closed liuche closed 7 years ago

liuche commented 7 years ago

Adds an event times function to the OpenHours class - it's a little hacky because it highly resembles getNextOpeningHours, but refactoring more didn't seem like it had clear benefits, and would touch a lot of code.

liuche commented 7 years ago

Yeah, I didn't feel like I wanted to refactor all that, and when I initially tried that approach, it involved de-private-ing a lot of functions that still didn't do exactly what we wanted. Tried to minimize code change here, but filing #618 to clean this up later. (Maybe next week, when we have LOTS OF TIME? :D)