Skip to content
This repository has been archived by the owner on Jul 10, 2020. It is now read-only.

Integrate Chart.java as dependency and remove ChartJs own wrappers from project #88

Open
dyorgio opened this issue Oct 30, 2018 · 2 comments

Comments

@dyorgio
Copy link

dyorgio commented Oct 30, 2018

Hello @moberwasserlechner,

I just found this very good project that already work on ChartJS Java wrappers, and keep it up-to-date with last chart.js distro.

How about reduce scope of this project to only use Chart.java (https://github.com/mdewilde/chart/) as dependency and make it works with Vaadin platform/components.

@moberwasserlechner
Copy link
Owner

Hi @dyorgio,

unfortunately this is not an option for me in the next few months. I simply do not have the time to do it or review such major overhaul, which changes the api completely.

Moreover I do not work with Vaadin in any of my projects.

Only if a customer contracts me for a paid Vaadin project I would as a service to the community dedicate more time to this project and do time consuming stuff like updating to Vaadin 10+ or working on refactorings like your's.

I hope you understand.

@dyorgio
Copy link
Author

dyorgio commented Nov 1, 2018

Tks for take a time to look that @moberwasserlechner !
Saddly Vaadin 10 choice to move to a web-component arch will decrease the activity of < vaadin 8 components :(.
My project is using vaadin 8 and compatible vaadin 7 package in production, and I have no plans to do a migration. Probably I will work on this issue (remove internal wrappers and use external dep) on my private branch. I will keep you tuned.

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

No branches or pull requests

2 participants