search

alejandro-du / crudui

Automatically generate CRUD-like Vaadin views for any Java Bean
https://vaadin.com/directory#!addon/crud-ui-add-on
Apache License 2.0
86 stars 54 forks source link

Bug when using crudui under app-layout (with minimal reproducible setup) #45

Closed jflamy closed 5 years ago

jflamy commented 5 years ago

Here is a minimal setup to recreate a strange interaction that only happens with crudui (and not with a normal grid). Since app-layout is likely what many people will be using ...

Steps to reproduce •Clone https://github.com/jflamy/bug_crud.git •Run mvn exec:java (or start org.ledocte.owlcms.Main from IDE) •Click on the "Test" button -- observe there is no error message •Refresh the page (navigating directly to the URL) •Observe that there is an error message (cannot read property 'map' of null)

alejandro-du commented 5 years ago

Is this still a bug? The linked repo doesn't exist anymore.

watho commented 5 years ago

I think it is fixed upstream: https://github.com/vaadin/vaadin-grid-flow/issues/515

alejandro-du commented 5 years ago

Great. I'm closing this for now.