Breaks /api calls out into seperate route

germanattanasio / visual-recognition-nodejs

DEPRECATED: this repo is no longer actively maintained

Apache License 2.0

361 stars 351 forks source link

Breaks /api calls out into seperate route #175

Closed kognate closed 8 years ago

kognate commented 8 years ago

Now the app.js file isn't the only place for the routes. This commit breaks out the api calls into a seperate route file.

germanattanasio commented 8 years ago

beautiful. I imagine that you tested classifying an image and creating a classifier and everything was working.

kognate commented 8 years ago

Works On My Machine. I'm also targeting all these changes into the sprint8 branch, which I'll setup to deploy to staging (but I want to talk to you about that first).

I don't want sprint8 merged into master, yet, though.

germanattanasio commented 8 years ago

do you want this PR to be merged ?

kognate commented 8 years ago

Yes, this PR can be merged (it's should be merged into sprint8).

On Mon, Jun 27, 2016 at 6:05 PM German Attanasio Ruiz < notifications@github.com> wrote:

do you want this PR to be merged ?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/watson-developer-cloud/visual-recognition-nodejs/pull/175#issuecomment-228890277, or mute the thread https://github.com/notifications/unsubscribe/AAB8wsNS5gu_Qix5aaOFnMb8uye-FjEUks5qQEklgaJpZM4I_iMH .