🐛(aliases) sort aliases by local part
sort aliases by local part to ease front-end interface
This commit is contained in:
committed by
Marie
parent
5feee53bdd
commit
01583ba94f
@@ -433,9 +433,10 @@ class AliasViewSet(
|
|||||||
lookup_field = "pk"
|
lookup_field = "pk"
|
||||||
permission_classes = [permissions.DomainResourcePermission]
|
permission_classes = [permissions.DomainResourcePermission]
|
||||||
serializer_class = serializers.AliasSerializer
|
serializer_class = serializers.AliasSerializer
|
||||||
queryset = (
|
queryset = models.Alias.objects.all().select_related("domain")
|
||||||
models.Alias.objects.all().select_related("domain").order_by("-created_at")
|
filter_backends = [filters.OrderingFilter]
|
||||||
)
|
ordering_fields = ["local_part"]
|
||||||
|
ordering = ["local_part"]
|
||||||
|
|
||||||
def get_serializer_context(self):
|
def get_serializer_context(self):
|
||||||
"""Extra context provided to the serializer class."""
|
"""Extra context provided to the serializer class."""
|
||||||
|
|||||||
@@ -59,4 +59,7 @@ def test_api_aliases_list__authorized_ok(role):
|
|||||||
f"/api/v1.0/mail-domains/{access.domain.slug}/aliases/",
|
f"/api/v1.0/mail-domains/{access.domain.slug}/aliases/",
|
||||||
)
|
)
|
||||||
assert response.status_code == status.HTTP_200_OK
|
assert response.status_code == status.HTTP_200_OK
|
||||||
|
assert response.json()["results"] == sorted(
|
||||||
|
response.json()["results"], key=lambda x: x["local_part"]
|
||||||
|
)
|
||||||
assert response.json()["count"] == 5
|
assert response.json()["count"] == 5
|
||||||
|
|||||||
Reference in New Issue
Block a user