Serialization of Spreadsheet objects. Why is it explicitely blocked? #4324
Closed
MarcinOrlowski
started this conversation in
General
Replies: 3 comments
|
I do not know all of the reasons, but we were getting lots of bug reports (see #1757, #1741, and #932), and possibly others; plus a concern that some of the properties were not serializable (resources, SimpleXMLElement, and maybe others). That said, I believe 1757 is no longer an issue because of the way Dynamic Arrays was implemented, and 1741 can be addressed with any of at least 3 totally trivial changes. I cannot speak to 932; I don't have a good way to test caching. Despite that, It may be time to revisit this decision. I will try to have a PR next week. |
0 replies
|
Thanks for the follow-up. If that restriction could be lifted, that would be a really great. Fingers crossed :) |
0 replies
|
Release 4.0.0 now allows this. |
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
There's a commit a7259de that explicitly prevents serialization of the
Spreadsheetobject. I have a code that uses legacy version and successfully serializes instances ofSpreadsheetfor ages, but I'd like to upgrade the package and as serialization is important part of the current flow, I currently stuck.@MarkBaker as you are the author of that commit, could you please cast some light about the reasons?
All reactions