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

Add ListSerializer.validate() #2168

Closed
LilyFoote opened this issue Dec 1, 2014 · 1 comment · Fixed by #2232
Closed

Add ListSerializer.validate() #2168

LilyFoote opened this issue Dec 1, 2014 · 1 comment · Fixed by #2232

Comments

@LilyFoote
Copy link
Contributor

LilyFoote commented Dec 1, 2014

In d2d7e1d Field.validate was removed. This means that ListSerializer.validate is no longer called in ListSerializer.run_validation. I can use ListSerializer.to_internal_value instead, but this feels very weird on a serializer.

@tomchristie tomchristie added this to the 3.0.1 Release milestone Dec 1, 2014
@tomchristie
Copy link
Member

tomchristie commented Dec 1, 2014

Should ListSerializer.validate exist?

Yes, I think it should.

@tomchristie tomchristie changed the title Should ListSerializer.validate exist? Add ListSerializer.validate() Dec 3, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants