fix: client route collisions

vaadin / flow

Vaadin Flow is a Java framework binding Vaadin web components to Java. This is part of Vaadin 10+.

Apache License 2.0

619 stars 167 forks source link

fix: client route collisions #20327

Closed caalador closed 3 weeks ago

caalador commented 3 weeks ago

Fix the client server route collision to not take into account client layouts as they are not route targets.

For instance

  {
    "route": "flow",
    "params": {},
    "children": [
      {
        "route": "hello-hilla",
        "menu": {
          "title": "Hello React in Flow Layout"
        },
        "title": "Hilla in Flow",
        "flowLayout": true,
        "params": {}
      }
    ]
  },

Collision on flow is wrong as the actual existing route is only flow/hello-hilla and thus should not match against a server route flow