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

Updating Wit API version (20160526 -> 20170307) #344

Merged
merged 1 commit into from
Oct 30, 2018

Conversation

Franck-Dernoncourt
Copy link
Contributor

No description provided.

@palikar
Copy link
Collaborator

palikar commented Oct 30, 2018

Hey @Franck-Dernoncourt,

this is a small change and I would be happy to merge it. Before that I should, however, ask - have you tested the Wit API with this change? Have you noticed any "buggy behavior" when the new API version is used?

@Franck-Dernoncourt
Copy link
Contributor Author

I haven't done any careful comparison between the two versions of the wit API.

@palikar
Copy link
Collaborator

palikar commented Oct 30, 2018

I've investigated the new API a little bit and everything seems OK. I think this is good to be merge. If there are some issues with the new API versions, we'll have to work them out as there are actually people (here) that would like to see the usage of the new API.

@palikar palikar merged commit f89256b into Uberi:master Oct 30, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants