1)Этой штуки нет по дефолту в .env.dist

Этой тоже COMMENTS_PAGINATE_BY
2)Зачем тебе эти комменты?

3)Нет запятой + это тоже надо из енва тянуть

4)Имхо в двух местах надо менять логкику поведение тулбара - эт плохо

5)Ну скапипаситил ты эт с доки, но комменты то удали...

6)Почему ты тут напрямую подключаешь RecipesListView в мейн урлы - так делать плохо

7)Имхо метод назван не понятно + в его название не сказано, что ты чет сетишь в кеш

8)По хорошему поля модели стоит писать: Model.field_name.field.name, то есть User.username.field.name - это поможет потом избежать ошибок, когда у тебя в моделе кто-то название филда поменяет+ еще плюшки есть

9)Во-первых, добавь в название класса User, во-вторых, зачем тебе нужен этот комент? он фактически повторяет название класса

10) а юзернейм тут может быть None?

11)if request.user.is_anonymous лучше, чем начало условия с not

12)Это надо ограничвать на уровне модели, а не формы

13)Не совсем понятно, почему бы это не вынести в сеттингс и импортить от туда?

14)А эт зачем? тебе же джанга не даст создать юзера, если юзернейм будет уже в бд так как поле uniq

15)Опять чет сомнительное

16)я конечно не уверен, но мне кажется, что ты какое-то дефолт поведение переписал

17)такое делать нельзя, солид нарушаешь

18) а какой сокральный смысл имеет это? там же и так будет user и users по дефолту, так как у тебя модель так называется

19) По код стайлу джанги: сначала идет Meta, потом str, а уже потом методы

https://docs.djangoproject.com/en/dev/internals/contributing/writing-code/coding-style/#model-style
20)Почему у тебя в одном классе есть Meta, а в другом нет? Какой смысл давать вербозы половине сущностей?
21)Почему ты это делаешь тут, если у тебя есть селери таска, которая делает то же самое?

В чем проблема ее тут заюзать?
22)Это пишется проще. return self.expiration < now()

23)Юзай менеджеры модели, чтобы у тебя не было куча мест с одинаковыми orm запросами
24)Зачем ты тут тайпхинт заюзал? У тебя их негде нет, но зато в сетапе тестов заюзал)

25)Очень плохая идея такие тесты делать, ты тестишь какой-то тайтл, который может менятся. Не тестируй сам контент страницы

26)Тестить то какой темплейт юзался - тоже странно

27)Зачем тебе эт надо было в отдельный метод выносить?

28)До тестов докапаться есть до чего конечно - но мне лень. То что их написал уже +rep
29)Выглидит не красиво, стоит делать в одном стиле все

30) Не понял зачем тебе декоратор, которым ты оборачиваешь классы, почему бы миксин не сделать?

31)Ты эту штуку пишешь в каждой вьюхе - время задуматься

32)Это какая-то логика с сессией точно не уровень абстракции метода form_valid

33)ок, сверх челу поможешь, мог еще написать что мб проблема в нем просто

34)Какой смысл это выносить отдельно, в чем проблема там сразу это сделать

35)какая-то сверх сложная бизнес логика, который не место во вьюхе

36) тут можно замашнить за харкод таймдельты

37)Не совсем понятно, зачем тебе is_valid_uuid тут делать

38)Пепка не любит такие переносы строк

39)Почему апи accounts и recipe в одном месте? Это два независимых приложения

40)а тут в теории может не быть username?)

41)Эт же не будет правильно работать, оно ж один раз в самом начале сделает запрос, а потом там будет устаревшая дата, или на это и был расчет?

(Мб я конечно чет путаю)
42)А еще эту штуку лучше делать через доп серилизатор
43)Какая-то сверх сильная система, мне кажется эт лучше проще делать

44)где-то я уже это видел) Дублировать код не оч классно

45)Во-первых, я эт опять же видел в каком-то похожем стиле + эта штука кучу запросов делает + не понятно ничего + куча бизнес логике там, где ее быть не должно

46)Никогда так не делай

47)Почему эти цифры такие? Откуда они?

48)немного не в тему, но почему у тебя все в кучу, почему оно лежит прямо в джанге?

49)Почему это тут, а не в менеджере модели?

50)Почему какие-то вьюхи функции, а какие-то классы? Мб стоит писать в одном стиле?
51)ПОЧЕМУ 60 * 60? Почему бы это не вынести куда-то в константы, тут ваще не понятно, что и зачем это

52)А и то и другое не может быть вместе?

53)Лямбда фор вот? Почему бы просто так не оставить Category.objects.all
54)Вот эт вот - это пиздец, а не функция, чтобы понять что тут происходит надо потратить кучу времени, ее надо раздрабить на функции. Плюс куча бизнес логики, которой тут тоже быть не должно

У тебя такой штуки очень много, где дохуя бизнес логики и прочих моментов во вьюшках. Так быть не должно
55)Куча моментов, где ты ебешь базу хз зачем. То есть делаешь сверх много запросов, когда эт не нужно.

Про запросы - там вроде есть моменты, где все будет ваще плохо, но мне было лень разворачивать проект и проверять эт все, так что советую: залить базу и идти в дебаг тулбаре посмотреть запросы на каждой вьюшке
56)Еще раз повторю - пиши запросы в менеджере модели. потому что сейчас вьюшки читать не реально.
57)Еще раз дубляж кода, эти функции - сверх одинаковые

58) Сверх странная логика в закладки рецепта добавлять/удалять юзера, чет тут не то

59)FormView спасет тебе тут жизнь

60)Совет: погугли про поетри

61)Почему у тебя бейз шаблон валяется в recipe?
Шаблоны я особо не чекал, так что про них наверное можно тоже чет посмотреть.
62)У тебя нигде не юзаются тайпхинты и тут ты решил еще и typing импортнуть, вай?

Плюс мне кажется, данный алго

С первого взгляда вроде все. Но скорее всего чет упустил.

Совет: перед тем как строчить код, как легенда, стоит чуток посидеть и подумать. Пытайся делать функции короче, разделяй уровень абстракции функций/классов/методов, чтобы у тебя не было кучи бизнес логики во вьюхах например.

Hi Team,
I have tried all my best and follow your steps from first but when I am tried to run "python manage.py makemigrations". I got error.

Thanks