-
Notifications
You must be signed in to change notification settings - Fork 44
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
Vaadin 7.3.0 support #37
Comments
+1, this addon doesn't work with 7.2.0 either... |
+1, I need a feature of Vaadin 7.2.0 so really would like an updated version |
Don't know where to best place this, but I've made a fork with the offending code removed. The Spring session annotation instead of @PreserveOnRefresh is probably not going to work anymore: |
I've updated the add-on. Versions 3.x support Vaadin 7.2.x. |
Hiya,
Thank you for a great add-on! Unfortunately there's been a change in the upcoming Vaadin release (7.3.0) that is incompatible with this add-on as it stands.
This is because there is no longer a VaadinSession.getCurrent().getPreserveOnRefreshUIs() -method present, I was able to make it work for me by just commenting out the whole bit that deals with that in SpringUIProvider.getUIClass() but not at all sure if that's just because this is meant to deal with a scenario which isn't present in my app or if Vaadin just deals with refresh-persistent UI's in a way now that doesn't require you to do this bit.
Thanks,
Peter
The text was updated successfully, but these errors were encountered: