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

Fix weatherThen #2

Open
cjmoeller opened this issue Aug 13, 2014 · 7 comments
Open

Fix weatherThen #2

cjmoeller opened this issue Aug 13, 2014 · 7 comments
Labels

Comments

@cjmoeller
Copy link
Member

throws an exception. will fix this soon.

@cjmoeller cjmoeller added the bug label Aug 13, 2014
@cjmoeller
Copy link
Member Author

Fixed the exception. But if you use "What is the weather on monday in frankfurt", both "weatherThen" and "weatherThenHere" are called.

@cjmoeller
Copy link
Member Author

It's a problem with the regexs

@jhbruhn
Copy link
Member

jhbruhn commented Aug 15, 2014

@cjmoeller could you check if it is fixed now?

@cjmoeller
Copy link
Member Author

Nope. Look at this: http://i.imgur.com/Z5SQyPZ.png . It should return null, because there's a place mentioned. At the moment it's like 'contains' but it has to be like 'equals'...

@jhbruhn
Copy link
Member

jhbruhn commented Aug 15, 2014

try regex.test, that's what the server is using. I think. Strawberry. Try it with the master, it should be working.

@cjmoeller
Copy link
Member Author

@jhbruhn
Copy link
Member

jhbruhn commented Aug 15, 2014

Weird

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

No branches or pull requests

2 participants