Skip to content
This repository has been archived by the owner on Aug 14, 2019. It is now read-only.

Updated Sample Project #7

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

kabirkhan
Copy link

Hi the sample was asking for the a LuisSubscriptionID. However, the API now requires a key from an azure deployment of a Cognitive Services LUIS API. I changed this wording to make it more clear to future developers.

Also, I updated the sample to use Android support 24.2.0, gradle 2.14.1 and to compile for SDK version 24. Everything else still works the same in the sample, it is now just easier to use.

@msftclas
Copy link

Hi @kabirkhan, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla.microsoft.com.

TTYL, MSBOT;

@msftclas
Copy link

@kabirkhan, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.

Thanks, MSBOT;

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

Successfully merging this pull request may close these issues.

3 participants