[#182] Добавить фильтр на списках "учитывать вклад пользователей Hexlet"

[ReYaNOW]

Сделал пока черновой вариант, чтобы уточнить, правильно ли я понял Issue #182 и сделал то, что требуется.
Фильтр работает при помощи поля "признак персонала" как предложил fey #182 (comment).
Фильтр добавил только в leaderboard, как писалось в ветке.
Запустить на render, к сожалению, не вышло.

[fey]

@ReYaNOW по коду не подскажу, а с деплоем, какие сложности?

[ReYaNOW]

@fey ==> Build successful 🎉
==> Deploying...
==> Using Node version 20.12.2 (default)
==> Docs on specifying a Node version: https://render.com/docs/node-version
==> Using Bun version 1.1.0 (default)
==> Docs on specifying a bun version: https://render.com/docs/bun-version
==> Running 'make start-production'
gunicorn -b 0.0.0.0:8000 config.wsgi:application
[2024-05-20 18:57:48 +0300] [85] [INFO] Starting gunicorn 20.1.0
[2024-05-20 18:57:48 +0300] [85] [INFO] Listening at: http://0.0.0.0:8000/ (85)
[2024-05-20 18:57:48 +0300] [85] [INFO] Using worker: sync
[2024-05-20 18:57:48 +0300] [87] [INFO] Booting worker with pid: 87
/opt/render/project/src/.venv/lib/python3.11/site-packages/django/db/models/fields/init.py:1535: RuntimeWarning: DateTimeField Contribution.created_at received a naive datetime (2023-06-01 00:00:00) while time zone support is active.
warnings.warn(
==> Your service is live 🎉
И после этого отдает пустую страницу и http 502
вот кстати сама страничка
https://hexlet-friends-reyan.onrender.com/

пробовал запуск и через тестовый сервер django и через gunicorn

[ReYaNOW]

В другой раз вот такую ошибку выдал
django.core.exceptions.SuspiciousFileOperation: The joined path (/images/og_images/default_og_image.jpg) is located outside of the base path component (/opt/render/project/src/static)
Пока не разбирался что ето

[fey]

Мне кажется, что вы не выполнили какие-то шаги. попробуйте пройтись по инструкции по деполю (должна быть в репе). тут у вас приложение детектится как nodejs

[ReYaNOW]

Врятли это проблема в установке, так как в 3 и 4 учебном проекте render писал тоже самое, но они работают

[ReYaNOW]

В итоге решил задеплоить на давно арендованном vps и проект запустился с пол пинка, без каких либо проблем. Может мощностей базового плана render не хватает, я не знаю.
вот он

Теперь осталось понять, сделал ли я то, что требовалось или нет :)

[fey]

@ReYaNOW давайте сделаем не в виде переключателя, а в виде селекта с поялми "с сотрудниками/без сотрудников/все"

[ReYaNOW]

Хорошо, переделаю

[fey]

@ReYaNOW как будете готовы к ревью, поменяйте состояние пулл реквеста и пингуйте. Не забудьте демку задеплоить, чтобы посмотреть визуально.

[ReYaNOW]

@fey Переделал, демку обновил, вроде работает
Заодно хотел спросить, локализацию лучше отдельным пул реквестом добавлять или нет? (не понял формулировку в README.md)

Так же вынес часть кода из вьюх, в которых я добавлял работу с новым фильтром в отдельный миксин, для уменьшения дубликации кода

[fey]

по i18n @sgmdlt подскажи нам.

[sgmdlt]

Локализацию можно этим же ПР. В редми уточнение, что если у вас не стоит задачи локализации, то не надо создавать тикеты с одной лишь локализацией без требования лишь чтобы перевести как-то иначе.

[sgmdlt]

Вы можете прислать как новый с локализацей, так и добавить в этот.

[ReYaNOW]

@sgmdlt Понял, спасибо за пояснение
Добавил локализацию

Жду ревью