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

[PiT 24.5] com.vaadin.flow.router.NotFoundException: No layout for client path '' #19956

Closed
ZheSun88 opened this issue Sep 16, 2024 · 1 comment · Fixed by vaadin/hilla#2733
Closed

Comments

@ZheSun88
Copy link
Contributor

ZheSun88 commented Sep 16, 2024

Description of the bug

when start a app using 24.5.0.alpha17, i saw this error in the build log and got a blank app

this error only happens with the latest alpha.. not in previous ones

com.vaadin.flow.router.NotFoundException: No layout for client path ''
        at com.vaadin.flow.router.internal.DefaultRouteResolver.resolve(DefaultRouteResolver.java:58) ~[flow-server-24.5.0.alpha20.jar:24.5.0.alpha20]
        at com.vaadin.flow.router.Router.resolveNavigationTarget(Router.java:120) ~[flow-server-24.5.0.alpha20.jar:24.5.0.alpha20]
        at com.vaadin.flow.component.UI.renderViewForRoute(UI.java:2009) ~[flow-server-24.5.0.alpha20.jar:24.5.0.alpha20]
        at com.vaadin.flow.component.UI.browserNavigate(UI.java:1877) ~[flow-server-24.5.0.alpha20.jar:24.5.0.alpha20]
        at com.vaadin.flow.component.ComponentEventBus.fireEventForListener(ComponentEventBus.java:239) ~[flow-server-24.5.0.alpha20.jar:24.5.0.alpha20]
        at com.vaadin.flow.component.ComponentEventBus.handleDomEvent(ComponentEventBus.java:488) ~[flow-server-24.5.0.alpha20.jar:24.5.0.alpha20]
        at com.vaadin.flow.component.ComponentEventBus.lambda$addDomTrigger$dd1b7957$1(ComponentEventBus.java:298) ~[flow-server-24.5.0.alpha20.jar:24.5.0.alpha20]
        at com.vaadin.flow.internal.nodefeature.ElementListenerMap.lambda$fireEvent$2(ElementListenerMap.java:475) ~[flow-server-24.5.0.alpha20.jar:24.5.0.alpha20]
        at java.base/java.util.ArrayList.forEach(ArrayList.java:1511) ~[na:na]

Expected behavior

the app works

Minimal reproducible example

curl -s -f 'https://start.vaadin.com/dl?&preset=react-crm-tutorial&projectName=react-crm-tutorial' -o react-crm-tutorial.zip
unzip react-crm-tutorial.zip
cd react-crm-tutorial
perl -0777 -pi -e 's|(<vaadin.version>)([^\s]+)(</vaadin.version>)|${1}24.5.0.alpha17${3}|g' pom.xml
perl -pi -e 's|(\s*)(</properties>)|$1$2\n$1<repositories><repository><id>v</id><url>https://maven.vaadin.com/vaadin-prereleases</url></repository></repositories>|' pom.xml
perl -pi -e 's|(\s*)(</properties>)|$1$2\n$1<pluginRepositories><pluginRepository><id>v</id><url>https://maven.vaadin.com/vaadin-prereleases</url></pluginRepository></pluginRepositories>|' pom.xml
perl -pi -e 's|<java.version>21</java.version>|<java.version>17</java.version>|' pom.xml
./mvnw

Versions

  • Vaadin / Flow version: 24.5.0.alpha17/24.5.0.alpha20
caalador added a commit to vaadin/hilla that referenced this issue Sep 16, 2024
Empty array should just return
and not wrap with empty array.

Fixes vaadin/flow#19956
@mshabarov
Copy link
Contributor

The same project gives me "Could not navigate to ''" for older alphas and even for 24.4.x.

@mshabarov mshabarov added the hilla Issues related to Hilla label Sep 16, 2024
platosha pushed a commit to vaadin/hilla that referenced this issue Sep 17, 2024
* fix: Do not wrap with layout for empty array

Empty array should just return
and not wrap with empty array.

Fixes vaadin/flow#19956

* Add test for no element for empty array

---------

Co-authored-by: Zhe Sun <[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 a pull request may close this issue.

2 participants