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

Refactor deserializers #321

Merged
merged 2 commits into from
Sep 29, 2016
Merged

Refactor deserializers #321

merged 2 commits into from
Sep 29, 2016

Conversation

BrianHicks
Copy link
Contributor

This removes the need to render things in serializers. See the updated module author's guide for how the struct tags are now used.

Next step: letting users user more complex values in params.

@BrianHicks BrianHicks force-pushed the feature/lists-in-params branch 2 times, most recently from 7617910 to 59703ad Compare September 28, 2016 19:12
This was referenced Sep 29, 2016
@BrianHicks BrianHicks added this to the 0.3.0 milestone Sep 29, 2016
@BrianHicks BrianHicks self-assigned this Sep 29, 2016
@BrianHicks
Copy link
Contributor Author

Just to track: this is a partial fix for #110

@BrianHicks BrianHicks merged commit a30e04c into master Sep 29, 2016
@BrianHicks BrianHicks deleted the feature/lists-in-params branch September 29, 2016 23:02
stevendborrelli added a commit that referenced this pull request Oct 5, 2016
stevendborrelli added a commit that referenced this pull request Oct 6, 2016
stevendborrelli added a commit that referenced this pull request Oct 11, 2016
stevendborrelli added a commit that referenced this pull request Oct 12, 2016
BrianHicks added a commit that referenced this pull request Dec 22, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant