spring-guides / gs-crud-with-vaadin

Creating CRUD UI with Vaadin :: Use Vaadin and Spring Data JPA to build a dynamic UI
https://spring.io/guides/gs/crud-with-vaadin
Apache License 2.0
108 stars 141 forks source link

Undefined method in Grid setColums() #2

Closed cynicbox closed 8 years ago

cynicbox commented 8 years ago

Next lines can fix it grid.addColumn("id"); grid.addColumn("firstName"); grid.addColumn("lastName");

gregturn commented 8 years ago

@cynicbox I don't get what the problem is. When I run "mvn clean spring-boot:run" the app runs as expected.

tnock commented 8 years ago

@gregturn same problem here. Using your setup i end up with a Grid class from vaadin-server-7.4.4.jar and that simply does not have a setColumns method. Solution of @cynicbox works. Anyway, nice tutorial :-)

cynicbox commented 8 years ago

Thanks god I'm sane ;) But @gregturn is right too. I cloned this repo and everything works fine. It is a mystery.

mstahv commented 8 years ago

I think the issue is you only added vaadin-spring-boot-starter, not the vaadin-bom overriding the Vaadin version in use? Are you using Maven or Gradle?

Maybe we should change the wording "Additionally, you might want to override the used Vaadin version..." to such that it suggests that more strongly. Or/and just update the vaadin-spring-boot-starter with the latest Vaadin version.

gregturn commented 8 years ago

@mstahv Well, if the issue is not reproducible, I suggest we close it. However, if you think using vaadin's BOM is standard procedure, would you like to add it to the maven and gradle builds and also document it in the guide?