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

Consider webpack for frontend assets #247

Open
parente opened this issue May 12, 2016 · 3 comments
Open

Consider webpack for frontend assets #247

parente opened this issue May 12, 2016 · 3 comments
Labels

Comments

@parente
Copy link
Member

parente commented May 12, 2016

As I watch all the bower components components scroll by when we install our extension, I wonder if we should be using webpack to simple it down to just one AMD JS file. Or, at the very least, filter out things we don't need when we build our pip package.

@dalogsdon
Copy link
Contributor

We could remove bower and have an npm script to webpack rather than bower install. It can get confusing having 2 dependency files (bower.json and package.json).

@jhpedemonte
Copy link
Collaborator

I'm down with this, although probably split into two:

  1. small JS file which initializes extension and updates menus
  2. larger JS file which is loaded on demand when user starts using dashboards

@parente
Copy link
Member Author

parente commented May 12, 2016

I'm leaving this out of 0.6.0 for now until we have a better sense of how much effort it will take to do the metadata switchover which is more important. If that goes smoothly and we have cycles, we can look at this too.

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

No branches or pull requests

3 participants