Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
acidmaksim opened this issue Jul 26, 2022 · 13 comments
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@acidmaksim
Copy link
Contributor

acidmaksim commented Jul 26, 2022

Make it possible to see the Leaderboard with and without all users, including Hexlet employees

@acidmaksim acidmaksim added enhancement New feature or request good first issue Good for newcomers labels Sep 12, 2022
@acidmaksim acidmaksim changed the title Сделать разные представления (без сотрудников Хекслета и с ними) Make different representations (without and with Hexlet staff) Sep 12, 2022
@SaltyFingers
Copy link

SaltyFingers commented Sep 13, 2022

Взял в работу
И сразу пара вопросов:

  1. Этот фильтр должен быть на главной или на вкладке "Контрибьюторы"?
  2. Не нашел, как отличить сотрудника от несотрудника.

@acidmaksim
Copy link
Contributor Author

Этот фильтр должен быть в LeaderBoard
@zhabinka Серег, поможешь со вторым вопросом?

@SaltyFingers
Copy link

@acidmaksim, вопрос. Пока не до конца могу понять, с какой стороны подойти к решению. Попробовал через django_filters, как в 4-ом проекте, не выходит. Попробовал через request брать состояние чекбокса - тоже. Можешь, пожалуйста, издалека подсказать, в какую сторону лучше думать?

@zhabinka
Copy link
Contributor

Не нашел, как отличить сотрудника от несотрудника.

Сейчас свойство Visible устанавливается для каждого участника отдельно. Хорошо бы сделать это через группы пользователей. Например, добавить новую сущность – Команды.

@zhabinka
Copy link
Contributor

@SaltyFingers, а какие проблемы с django_filters?

@SaltyFingers
Copy link

SaltyFingers commented Oct 4, 2022

@zhabinka, привет, прошу прощения, что долго. В общем я сделал так:

  • создал форму StaffFilter(django_filters.FilterSet) с BooleanFilter чекбоксом
  • засунул эту форму в шаблон
  • во Home вьюхе объявил filterset_class
  • contributors_for_month теперь ПО ИДЕЕ содержит в себе queryset после фильтрации

В итоге получаю ошибку Module "contributors.admin.custom" does not define a "CustomAdminSite" attribute/class
Это последнее, к чему пришел. До этого тыкался, все сводилось к тому, что чекбокс ничего не делал.

upd: последняя проблема сама по себе исчезла

@acidmaksim
Copy link
Contributor Author

@zhabinka подскажешь?

@SaltyFingers
Copy link

Освободил задачу для желающих

@LilDrugHill
Copy link

Возбмусь. Так как обозначены сотрудники и не-сотрудники.
Если реализовать "команды" то это нужно изменить процесс регистрации или добавить возможность выбора команды. Поясните момент

@acidmaksim
Copy link
Contributor Author

Задача свободна

@LilDrugHill
Copy link

Снова беру

@fey fey changed the title Make different representations (without and with Hexlet staff) Добавить фильтр на списках "учитывать вклад пользователей Hexlet Aug 25, 2023
@fey
Copy link
Collaborator

fey commented Mar 4, 2024

Сейчас в админке есть поле "признак персонала". Предлагаю фильтровать по этому полю.

@ReYaNOW
Copy link
Contributor

ReYaNOW commented May 17, 2024

Взял в работу

sgmdlt added a commit that referenced this issue May 29, 2024
[#182] Добавить фильтр на списках "учитывать вклад пользователей Hexlet"
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
Status: 🔖 Ready
Development

No branches or pull requests

6 participants