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

Relative Reference of rest.js in client > js > app > modules > persistence > persistence.js #25

Closed
shabeermothi opened this issue Aug 22, 2015 · 1 comment

Comments

@shabeermothi
Copy link

Was setting up the keen data explorer using the instructions in readme and encountered an issue while serving the project with gulp.. Below is the trace

[00:38:08] Using gulpfile /Users/shabeermothi/projects/keen-data-explorer/explorer/gulpfile.js [00:38:08] Starting 'build-scripts'... [00:38:08] Starting 'images'... [00:38:08] Starting 'fonts'... [00:38:08] Starting 'build-styles'... [00:38:08] Starting 'connect'... [00:38:08] Server started http://localhost:8082 [00:38:08] Finished 'connect' after 238 ms [00:38:08] Starting 'watch'... [00:38:08] Finished 'watch' after 135 ms Autoprefixer's process() method is deprecated and will removed in next major release. Use postcss([autoprefixer]).process() instead [00:38:08] Finished 'build-styles' after 820 ms [00:38:08] Finished 'images' after 834 ms [00:38:10] Finished 'fonts' after 2.2 s events.js:85 throw er; // Unhandled 'error' event ^ Error: module "./rest.js" not found from "/Users/shabeermothi/projects/keen-data-explorer/explorer/client/js/app/modules/persistence/persistence.js" at notFound (/Users/shabeermothi/projects/keen-data-explorer/explorer/node_modules/browserify/index.js:812:15) at /Users/shabeermothi/projects/keen-data-explorer/explorer/node_modules/browserify/index.js:762:23 at /Users/shabeermothi/projects/keen-data-explorer/explorer/node_modules/browserify/node_modules/browser-resolve/index.js:250:24 at /Users/shabeermothi/projects/keen-data-explorer/explorer/node_modules/browserify/node_modules/browser-resolve/node_modules/resolve/lib/async.js:55:18 at load (/Users/shabeermothi/projects/keen-data-explorer/explorer/node_modules/browserify/node_modules/browser-resolve/node_modules/resolve/lib/async.js:69:43) at onex (/Users/shabeermothi/projects/keen-data-explorer/explorer/node_modules/browserify/node_modules/browser-resolve/node_modules/resolve/lib/async.js:92:31) at /Users/shabeermothi/projects/keen-data-explorer/explorer/node_modules/browserify/node_modules/browser-resolve/node_modules/resolve/lib/async.js:22:47 at FSReqWrap.oncomplete (fs.js:95:15)

Found the rest.js module is required from persistence.js (client > js > app > modules > persistence > persistence.js) as a reference from the same path whereas it should be a folder up in the hierarchy.

Current reference in persistence.js
REST: require('./rest.js')

Actual reference in persistence.js which would make it work
REST: require('../rest.js')

The data explorer is cool btw 👍 and the readme is very much descriptive 👍

josephwegner pushed a commit that referenced this issue Aug 28, 2015
@dustinlarimer
Copy link
Contributor

@shabeermothi @clayallsopp we've recently migrated from this intermediary persistence model to API-backed persistence, handled by the Saved Query resource. We'll be making a formal announcement about this soon, but in the meantime feel free to hit us up with any questions or concerns! Thanks!

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

No branches or pull requests

2 participants