Add format argument to UUIDField
#2788
Conversation
|
LGTM |
|
Seconded here. |
docs/api-guide/fields.md
Outdated
To allow serializations to control the representation format of a UUID value
|
+1 |
There was a problem hiding this comment.
Are we always going to be handling the int case correctly here? Any cases where that can just be an int, rather than a UUID instance?
There was a problem hiding this comment.
I'm not entirely sure what potential problem you foresee here.
Which int case? The case where format is 'int', or the case where an int has somehow been passed to to_representation? None of the other fields check too rigorously for an incorrect type being passed to to_representation.
|
One query worth having a quikc think about but otherwise reckon I'm happy for this to go in. |
There was a problem hiding this comment.
Could we have examples here?
|
Okay one last comment regarding the docs. Otherwise good after that! :) |
|
Closing this in favor of #3000. |
xordoquy
changed the title
format argument to UUIDField
|
@ovangle thanks for the PR. |
|
That's OK. I seem to have missed a couple of emails telling me to follow up on it. Sorry about that. |
|
@ovangle no problem at all |
5 participants
To allow serializations to control the representation format of a
UUID value