order_by property

[tpeaton]

This adds an 'order_by' property that can be overriden to specify a custom ordering for tags. The default is ['id'].

Example:

class SortedTagSerializer(TagListSerializerField):
    order_by = ['name']


class SomeSerializer(TaggitSerializer, ModelSerializer):
    tags = SortedTagSerializer()

I haven't been able to get the tests to run locally yet as outlined in the docs. (But it looks like they worked fine on Travis)

[glemmaPaul]

Thanks for the PR @tpeaton, I would love to add this option, but I don't think a default to id is a good idea. This may be because people already order on something else. What do you think?

[tpeaton]

Yea, I have mixed feelings about it as well. I was trying to avoid it, but it probably makes more sense to do an if block and only apply the ordering if order_by is specified. If you agree, I'll push up that change.

[glemmaPaul]

@tpeaton That sounds like a good idea yes! Could you (not necessarily needed) rebase the PR to one commit too?

[tpeaton]

Sure thing, thanks.

[glemmaPaul]

Merging this in, will be in the next release! Thanks for the great contribution! 👍

[tpeaton]

Awesome, thanks!