Skip to content
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

fix issue #51 #58

Merged
merged 6 commits into from
Jul 4, 2023
Merged

fix issue #51 #58

merged 6 commits into from
Jul 4, 2023

Conversation

dominik42
Copy link
Contributor

  • compile with Vaadin 24.1.0
  • using ColumnPathRenderer explicitely
  • extend exception log with key for column with missing value

@dominik42
Copy link
Contributor Author

The code format settings are still modified by my IDE, despite my 2 additional commits. I assume, this was done during the commit phase, so sorry for that. I do not want to alter your file/code format settings.

@javier-godoy javier-godoy self-assigned this Jun 26, 2023
@javier-godoy javier-godoy added this to To Do in Flowing Code Addons (legacy) via automation Jun 26, 2023
@javier-godoy
Copy link
Member

Thanks for your contribution. We will merge it to a new major version of the add-on, targeting Vaadin 24 (The current version attempts to be compatible with "23.1+" and it's quite difficult to implement such low level hacks while supporting a broad range of framework versions).

@javier-godoy javier-godoy changed the base branch from master to 2.0.0-wip July 4, 2023 17:25
@javier-godoy javier-godoy merged commit 2713546 into FlowingCode:2.0.0-wip Jul 4, 2023
1 check failed
Flowing Code Addons (legacy) automation moved this from To Do to Done Jul 4, 2023
@javier-godoy
Copy link
Member

Thanks again! The fix has been released in version 2.0.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

None yet

2 participants