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.

Sign up for GitHub

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

Jump to bottom

Update recommend api function usage based on changes in voyager #4

Open

ajainarayanan wants to merge 3 commits into vega:master from ajainarayanan:feature/ui-fix-compasssql-recommend

ajainarayanan commented Dec 4, 2017

This is in relation to this PR: vega/voyager-electron#32.

Updates the /recommend endpoint to use the right fetchCompassQLRecommend api function

Note:

Please do not merge this PR yet. The voyager node module still points to a local npm module.

ajainarayanan added 2 commits

December 3, 2017 23:35


          Fixes recommend endpoint to use latest datavoyager api function

f4288e1


          Adds build/server + yarn.lock files

499a115

ajainarayanan mentioned this pull request

Fix dependencies based voyager build change vega/voyager-electron#32

Open


          Removes unused Data type from routes.ts

172560e

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet