Solution

[raychw]

No description provided.

[Dimosphen1]

Make sure that all checks pass. Several changes were requested

[Dimosphen1]

Do not redefine an existing migration, create a new one instead

[Dimosphen1]

Use raise_exception=True here and in the other places as well to remove the if on condition while validating serializer

[vsmutok]

1. Avoid using an if condition to check if a serializer is valid. Instead, use the raise_exception=True flag when calling serializer.is_valid(). This will automatically raise a ValidationError if the data is invalid, which is then caught by the DRF exception handler to return a 400 Bad Request response.

Good example:

if request.method == 'POST':
    serializer = MovieSerializer(data=request.data)
    serializer.is_valid(raise_exception=True)
    serializer.save()
    return Response(serializer.data, status=status.HTTP_201_CREATED)

Bad example:

if request.method == 'POST':
    serializer = MovieSerializer(data=request.data)
    if serializer.is_valid():
        serializer.save()
        return Response(serializer.data, status=status.HTTP_201_CREATED)
    return Response(serializer.errors, status=status.HTTP_400_BAD_REQUEST)

[viktoria-rybenchuk]

LGTM!