Agreed, the people listing should be alphabetical on the person's name attribute. However, the problem is that I haven't quite figured where it would make the most sense to define a default ordering for something like a person set. In the current design, it could make some sense to define the default ordering as a class attribute in the PersonSet which would be checked by the base SearchSet class when building the query. It could also make some sense to define the ordering as a default value in the constructor of the PersonSearchParams class instead of letting the base SearchSetParams set the value to None. Both solutions would work but not sure which might be more desirable.
Agreed, the people listing should be alphabetical on the person's name attribute. However, the problem is that I haven't quite figured where it would make the most sense to define a default ordering for something like a person set. In the current design, it could make some sense to define the default ordering as a class attribute in the PersonSet which would be checked by the base SearchSet class when building the query. It could also make some sense to define the ordering as a default value in the constructor of the PersonSearchParams class instead of letting the base SearchSetParams set the value to None. Both solutions would work but not sure which might be more desirable.