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

Produce development version to work with cylc-jupyterhub #36

Closed
kinow opened this issue Feb 26, 2019 · 6 comments
Closed

Produce development version to work with cylc-jupyterhub #36

kinow opened this issue Feb 26, 2019 · 6 comments
Assignees
Labels
help wanted Extra attention is needed
Milestone

Comments

@kinow
Copy link
Member

kinow commented Feb 26, 2019

Now cylc-jupyterhub supports pointing to any directory with an index.html to serve as the Web GUI for the Cylc UI Server.

It would be good to have a better way to code cylc-web and see the results with and without the Hub. Preferably something quick, as generating production files takes some 20 seconds that could be better spent doing something else.

@kinow
Copy link
Member Author

kinow commented Feb 26, 2019

Current overview of modules in the application.

image

Also adding the task to produce this report.

Later we may have to review the libraries we are using and try to reduce some. Especially after adding the suite graph, which may impact the project performance.

asset size limit: The following asset(s) exceed the recommended size limit (244 KiB).
This can impact web performance.
Assets: 
  fonts/materialdesignicons-webfont.d15c1216.ttf (407 KiB)
  fonts/materialdesignicons-webfont.abb4e749.eot (407 KiB)
  img/materialdesignicons-webfont.ae4dbede.svg (3.05 MiB)
  img/do.88a82fcc.svg (443 KiB)
  img/do.704fd2a9.svg (456 KiB)
  css/chunk-vendors.c9a2df19.css (408 KiB)
  js/chunk-vendors.993984f7.js (1.01 MiB)

 warning  

entrypoint size limit: The following entrypoint(s) combined asset size exceeds the recommended limit (244 KiB). This can impact web performance.
Entrypoints:
  app (1.47 MiB)
      css/chunk-vendors.c9a2df19.css
      js/chunk-vendors.993984f7.js
      css/app.852df6eb.css
      js/app.90b7a101.js

@kinow
Copy link
Member Author

kinow commented Feb 26, 2019

One alternative would be running the WebPack Proxy https://vuejs-templates.github.io/webpack/proxy.html, which is normally used to send the requests back to a running instance of the backend.

The issue is that we have Tornado handling authentication, cookie handling, and loading a static index.html. 🤔

@kinow kinow self-assigned this Feb 26, 2019
@kinow kinow added enhancement help wanted Extra attention is needed labels Feb 26, 2019
@kinow
Copy link
Member Author

kinow commented Feb 26, 2019

This option might work: vuejs/vue-cli#1317

If we are able to produce an index.html with the watch feature, that would be a good development setup IMO.

@kinow
Copy link
Member Author

kinow commented Feb 26, 2019

Well, apparently that works. Takes about 2 seconds, and produces a development version of index.html. Whew. Thought would have to dig into webpack, vue, and start trying to work on it from scratch :-) ❤️ OpenSource

@hjoliver
Copy link
Member

Sounds cool, might get you to explain this to me to tomorrow!

@kinow
Copy link
Member Author

kinow commented Feb 27, 2019

Sure thing! I'm hoping to have written a unit test, and then the #33 issue will be pretty much done (finally!)

@kinow kinow added this to the 0.1 milestone Sep 10, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants