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

fix: Layout should be collected as entyrpoint #20076

Merged
merged 1 commit into from
Sep 27, 2024

Conversation

caalador
Copy link
Contributor

Layout annotated classes should be
collected as entrypoints for
production builds.

Fixes #20074

Layout annotated classes should be
collected as entrypoints for
production builds.

Fixes #20074
@caalador caalador force-pushed the issues/20074-layout-entrypoint branch from 5473f90 to 586d0b9 Compare September 27, 2024 11:07
Copy link

sonarcloud bot commented Sep 27, 2024

Copy link

github-actions bot commented Sep 27, 2024

Test Results

1 137 files  ± 0  1 137 suites  ±0   1h 9m 3s ⏱️ -55s
7 400 tests + 1  7 350 ✅ + 1  50 💤 ±0  0 ❌ ±0 
7 767 runs  +24  7 707 ✅ +21  60 💤 +3  0 ❌ ±0 

Results for commit 586d0b9. ± Comparison against base commit 4844adb.

♻️ This comment has been updated with latest results.

Copy link
Contributor

@mshabarov mshabarov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works as expected after this patch and also I don't see messages like:

2024-09-27T15:11:15.928+03:00 ERROR 10241 --- [nio-8080-exec-8] c.v.flow.component.internal.UIInternals  : The component class com.vaadin.flow.component.applayout.DrawerToggle includes '@vaadin/app-layout/src/vaadin-drawer-toggle.js' but this file was not included when creating the production bundle. The component will not work properly. Check that you have a reference to the component and that you are not using it only through reflection. If needed add a @Uses(DrawerToggle.class) where it is used.

anymore.
LGTM.

@mshabarov mshabarov merged commit d91be76 into main Sep 27, 2024
26 checks passed
@mshabarov mshabarov deleted the issues/20074-layout-entrypoint branch September 27, 2024 12:25
vaadin-bot pushed a commit that referenced this pull request Sep 27, 2024
Layout annotated classes should be
collected as entrypoints for
production builds.

Fixes #20074
vaadin-bot added a commit that referenced this pull request Sep 27, 2024
Layout annotated classes should be
collected as entrypoints for
production builds.

Fixes #20074

Co-authored-by: caalador <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

@Layout components are not included in the production bundle
3 participants