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.

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

Building caravel from master throws error in javascript console #980

Closed
shkr opened this issue Aug 18, 2016 · 3 comments · Fixed by #990
Closed

Building caravel from master throws error in javascript console #980

shkr opened this issue Aug 18, 2016 · 3 comments · Fixed by #990

Comments

@shkr
Copy link
Contributor

shkr commented Aug 18, 2016

dashboard.entry.js:71 Uncaught Error: Cannot find module "react-map-gl"
@xrmx
Copy link
Contributor

xrmx commented Aug 18, 2016

What command is "building"? :)

@shkr
Copy link
Contributor Author

shkr commented Aug 18, 2016 •
edited
Loading

Oops. I meant after building. This is the sequence of commands.

pushd caravel/assets
npm install
popd
python setup.py sdist
pip install dist/caravel-0.10.0.tar.gz
caravel runserver -d

When I visit the dashboard localhost:8100/caravel/dashboard/4/ on caravel, this error is thrown in the javascript console.

dashboard.entry.js:71 Uncaught Error: Cannot find module "react-map-gl"

@mistercrunch
Copy link
Member

You forgot npm run prod after npm install

@mistercrunch mistercrunch mentioned this issue Aug 19, 2016
Merged
zhaoyongjie pushed a commit to zhaoyongjie/incubator-superset that referenced this issue Nov 17, 2021
@nytai @zhaoyongjie
refactor(SupersetClient): use new /api/v1/security/csrf_token/ endpoi…
c21d706 
…nt (apache#980)
zhaoyongjie pushed a commit to zhaoyongjie/incubator-superset that referenced this issue Nov 24, 2021
@nytai @zhaoyongjie
refactor(SupersetClient): use new /api/v1/security/csrf_token/ endpoi…
2a7e865 
…nt (apache#980)
zhaoyongjie pushed a commit to zhaoyongjie/incubator-superset that referenced this issue Nov 25, 2021
@nytai @zhaoyongjie
refactor(SupersetClient): use new /api/v1/security/csrf_token/ endpoi…
f720ebd 
…nt (apache#980)
zhaoyongjie pushed a commit to zhaoyongjie/incubator-superset that referenced this issue Nov 26, 2021
@nytai @zhaoyongjie
refactor(SupersetClient): use new /api/v1/security/csrf_token/ endpoi…
01b7054 
…nt (apache#980)
@laxmanraparthi laxmanraparthi mentioned this issue Jul 7, 2023
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Documenting making your own build mistercrunch/superset
3 participants
@xrmx @mistercrunch @shkr