Solution #761
Solution #761
Conversation
There was a problem hiding this comment.
Great job on implementing the models, serializers, views, and URL configurations! 🎉 Your code meets all the task requirements and constraints effectively. While your serializers are correctly implemented, consider using serializers.ModelSerializer for MovieSerializer and CinemaHallSerializer in the future to reduce boilerplate code and enhance maintainability. Keep up the excellent work! 😊
✨ Please rate this review in the Platform's notification center. I will be grateful for your feedback! ✨
| from rest_framework import serializers | ||
|
|
||
| from cinema.models import Movie | ||
| from cinema.models import Movie, Genre, Actor, CinemaHall | ||
|
|
||
|
|
||
| class MovieSerializer(serializers.Serializer): |
There was a problem hiding this comment.
Consider using serializers.ModelSerializer for MovieSerializer to simplify the code. This will automatically generate fields based on the model definition and reduce redundancy.
| return instance | ||
|
|
||
|
|
||
| class CinemaHallSerializer(serializers.Serializer): |
There was a problem hiding this comment.
Consider using serializers.ModelSerializer for CinemaHallSerializer to simplify the code. This will automatically generate fields based on the model definition and reduce redundancy.
No description provided.