-
Notifications
You must be signed in to change notification settings - Fork 40
Conversation
I may have been doing something wrong because |
I reduced the file size down to 1.7MB by not including the heavy .js.map file, fixed in 55a6a3d |
Sourcemaps are good if you're opening browser DevTools with production site, and they are not downloaded until then. But yeah, we don't need then as part of the bundle. |
If you need the .map files for debugging purposes, please see section about running development release, which serves all files from the yarn build, when a browser tries to download them. |
Awesome! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm happy with this PR and will merge. Thanks!
BTW, please check if fallback to index.html works as expected (see 2c9a53e).
Resolve #10
Introduces new UI. I've updated only
Makefile
to build new UI and move output toapps/epl/priv/htdocs
. This waybootsatrap
script can include it in bundle and application can serve static files. Only thing I have problem setting up is fallback to/index.html
when route not found. This should be handled by frontend and server should redirect everything to/index.html
/ cc @michalslaski @arkgil