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

made entry, dataset and chartdata serializable #2797

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

jkwiecien
Copy link

Made model base objects ChartData, Entry and DataSet serializable hopefully covering the whole model this way

@almic
Copy link
Contributor

almic commented Oct 28, 2018

Sorry if I'm not getting it, but is this really all that's needed for such a thing to work? I would need to test this before pulling it, and right now I'm working to bring the issue and pull request count down to a manageable number. I'll be back in a day or two and update you.

@almic almic added the enhancement improves upon existing functionality label Oct 28, 2018
@trav3711
Copy link

Hello! I'm currently trying to serialize an ArrayList of BarEntry objects and I keep getting an error. Is an Entry Serializable?

@hannesa2
Copy link
Contributor

While you wait 7 years for a merge (the last merge was 3 years ago), I merged it here already

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

Successfully merging this pull request may close these issues.

4 participants