-
Notifications
You must be signed in to change notification settings - Fork 6
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
feat(client): reduce the main bundle size #2818
Conversation
You can access the deployment of this PR at https://renku-ci-ui-2818.dev.renku.ch |
This fixes #2346 |
b723b56
to
719cf75
Compare
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.
Great improvement! 🙌
Some tests are failing. Feel free to remove any legacy Jest test trying to load a component in the DOM. It would be best if the same elements are tested at least once with Cypress or in Storybook
|
It seems that jest cannot deal with <Suspense> so we remove the corresponding tests.
9c77b4f
to
9fe767c
Compare
9fe767c
to
cfec556
Compare
Note about the removed tests: it seems that jest tests are not working in CI/CD with . The corresponding components should be covered by Cypress tests. |
Tearing down the temporary RenkuLab deplyoment for this PR. |
Fixes #2346.
Details:
/deploy renku=auto-update/renku-ui-3.13.0 extra-values=notebooks.userSessionPersistentVolumes.enabled=true
Before this PR
After this PR
Lighthouse reports
Before:
After: