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

Develop #106

Closed frialey63 closed 2 years ago

frialey63 commented 2 years ago

Hi Alejandro, Great component, thanks! I have a few minor patches for your consideration. The biggest thing I noticed (and fixed?!?) was that success notifications were being displayed even when a CrudOperationException was thrown from my add/update/delete operations. Also, I have added a ENTER shortcut for the default form and a method on GridCrud to add an update button in a column on the grid. My changes are in three separate commits with associated messages. Paul

alejandro-du commented 2 years ago

Thanks a lot for your contribution.