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

Serializer/deserializer overhaul #1317

Open
jasongrout opened this issue Apr 27, 2017 · 1 comment
Open

Serializer/deserializer overhaul #1317

jasongrout opened this issue Apr 27, 2017 · 1 comment
Milestone

Comments

@jasongrout
Copy link
Member

Currently the signature of Javascript serializers is confusing (they are called with a second argument that is different depending on who is calling them). Additionally, we have an instance serialize and a static serialize. #1300 addresses some of this, but we should get a clean consistent picture of serializers and deserializers for 7.0.

I'd like whatever we come up with to address:

  1. transferring information by transferring a delta of the existing attribute (which means that the deserializer needs to have access to the current state, which means the deserializer probably can't be called from a static method)
  2. Consistent signature for the serializer/deserializer
@jasongrout jasongrout added this to the 7.0 milestone Apr 27, 2017
@jasongrout
Copy link
Member Author

Moving to Future - this is a trickier issue than I think we want to tackle for 7.0.

CC @SylvainCorlay, @maartenbreddels.

@jasongrout jasongrout modified the milestones: Future, 7.0 May 17, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant