Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
pavlopro created a review comment on a pull request on mate-academy/py-movie-moviesession-cinemahall
It is a good practice to include one migration per app per PR in Django projects since it helps to avoid migrations folder overload in the future. Represent all changes done to models in one migrat...
pavlopro created a review comment on a pull request on mate-academy/py-movie-moviesession-cinemahall
`movie_session, not movie`
pavlopro created a review comment on a pull request on mate-academy/py-movie-moviesession-cinemahall
use get_movie_session_by_id method
pavlopro created a review comment on a pull request on mate-academy/py-movie-moviesession-cinemahall
use `get_movie_session_by_id` method
pavlopro created a review comment on a pull request on mate-academy/py-movie-moviesession-cinemahall
simplify to: .... ``` cinema_hall_id=cinema_hall_id, movie_id=movie_id ```
pavlopro created a review comment on a pull request on mate-academy/py-movie-moviesession-cinemahall
``` if session_date: queryset = queryset.filter(show_time__date=session_date ```
pavlopro created a review comment on a pull request on mate-academy/py-movie-moviesession-cinemahall
Add related names for relational fields
pavlopro created a review comment on a pull request on mate-academy/py-movie-moviesession-cinemahall
Add related names for relational fields
pavlopro created a review comment on a pull request on mate-academy/py-can-access-google-page
Please use parametrize decorator and add cases where valid_url and connection are both False and True
pavlopro created a review comment on a pull request on mate-academy/py-distance-class
`if isinstance(other, (float, int)): `
pavlopro created a review comment on a pull request on mate-academy/py-taxi-service-search-and-tests
This should be in `includes` folder
pavlopro created a review comment on a pull request on mate-academy/py-cinema-full-stack
Are you sure that this works?
pavlopro created a review comment on a pull request on mate-academy/py-herbivores-and-carnivores
`... and not herbivore.hidden`
pavlopro created a review comment on a pull request on mate-academy/py-herbivores-and-carnivores
You should place this check inside first `if` statement
pavlopro created a review comment on a pull request on mate-academy/py-taxi-service-search-and-tests
Create a separate template for your search form and include it where you need