Solution #708
Solution #708
Conversation
| @property | ||
| def full_name(self): | ||
| return f"{self.first_name} {self.last_name}" | ||
|
|
||
| def __str__(self): | ||
| return self.first_name + " " + self.last_name |
There was a problem hiding this comment.
You can use full_name property in this method: return self.full_name.
| router.register("genres", GenreViewSet, basename="genres") | ||
| router.register("actors", ActorViewSet, basename="actors") | ||
| router.register("cinema_halls", CinemaHallViewSet, basename="cinema_halls") | ||
| router.register("movies", MovieViewSet, basename="movies") | ||
| router.register( | ||
| "movie_sessions", | ||
| MovieSessionViewSet, | ||
| basename="movie_sessions" | ||
| ) |
There was a problem hiding this comment.
Keep router registrations consistent - use single-line format for all, or multi-line format for all.
Currently, movie_sessions differs from others.
There was a problem hiding this comment.
Great job on the implementation! 🎉 The code is well-structured and meets most of the task requirements. There are a few minor issues to address in the future, such as adding type annotations to class methods and avoiding unnecessary use of .all() before .prefetch_related() and .select_related(). Also, remember to exclude unrelated migration files from the task. Keep up the good work and continue refining your skills! 😊
✨ Please rate this review in the Platform's notification center. I will be grateful for your feedback! ✨
No description provided.