Jonathan Bardo
Jonathan Bardo
@EvanHerman I feel we might be too restrictive. What about we propose a free text alternative on every field (that way if they close for lunch they can manually add...
@fjarrett I agree, I wouldn't make it default for sure but I want to be as inclusive as we can. The use case we solved won't work for most European...
@EvanHerman I know a lot of European countries where they close during lunch time or are totally closed in August for example. With the current implementation it would be impossible...
@EvanHerman Travis seems to be failing https://travis-ci.org/godaddy/wp-contact-widgets/jobs/183766078. `store_closed` var is never used in admin.js line 188
@EvanHerman Sometimes Selenium is acting weird. When only 1 builds fails I normally try to restart that particular build. I will find some time to look into why it's acting...
@fjarrett That UI does look pretty cool. Adding multiple time block would definitely give people more flexibility.
@fjarrett Definitely. I would still keep the possibility to add free text at the end though.
I have the same problem when I install vassh with brew, but not when I clone this repo and source it in my bash_profile