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

Заполнение БД реальными данными, ошибка создания представителя команды #572

Closed
KonstantinRaikhert opened this issue Oct 8, 2024 · 4 comments · Fixed by #577
Assignees
Labels
bug Something isn't working

Comments

@KonstantinRaikhert
Copy link
Member

KonstantinRaikhert commented Oct 8, 2024

Где был обнаружен баг ?

При помощи команды export-db был создан dump файл. Но почему то к каждой команде привязался куратор в лице суперпользователя. Поэтому теперь невозможно создать нового представителя команды, т.к. уже к ней привязан Админ Админ.
Такое же поведение и с командой fill-db

Скриншоты

2024-10-08_17-16
2024-10-08_17-24
2024-10-08_17-27

Предложения и предположения как исправить

Возможно, необходимо пересмотреть модели Curator и User. Необходимо выяснить, почему так происходить.
Стоит еще посмотрь в сторону ролей, в форме понятно, почему не отображается Админ А. Но вопрос, почему он присваивается как куратор (представитель команды) по умолчанию.

@KonstantinRaikhert KonstantinRaikhert added the bug Something isn't working label Oct 8, 2024
@Toksi86
Copy link
Contributor

Toksi86 commented Oct 10, 2024

При заполнении базы данных командой fill-db используется цикл, где явно указывается, что куратор для каждый команды это пользователь с ID - 1, а это и есть наш суперпользователь.
for item in data:
if table_name == "main_team":
team_data = {
"id": item["id"],
"name": item["name"],
"city_id": item["city_id"],
"discipline_name_id": item["discipline_name_id"],
"curator_id": 1,
}
Поэтому он и представитель команды по умолчанию. Какое поведение должно быть в итоге?

Почему он не высвечивается в форме еще не разобрался, смотрю.

@Toksi86
Copy link
Contributor

Toksi86 commented Oct 10, 2024

Тест-куратора
Сделал, чтобы по умолчанию присваивалось значение None, так у всех команд больше не было куратора по умолчанию. Создал нового пользователя и назначил его куратором команды, после чего у команды появился куратор в поле, что видно на скриншоте. То есть логика отображения куратора, если он имеется тоже корректная.

Тест-куратора2
Также появился выбор из пользователей, которые имеют роль куратора. Что видно на скриншоте 2.

Тест-куратора3
И есть возможность его применить, после применения и обновления страницы команды, он сохраняется в форме.
Что видно на скриншоте 3.

Жду ответа о требуемой логике, оставить None в поле при выполнении команды import-db? При экспорте соответственно он забирал данные, где был админ и сохранял их, насколько я понял. Вытекающая проблема из первой, которая по сути решена. Жду ответа и делаю PR.

@KonstantinRaikhert
Copy link
Member Author

@Toksi86 Думаю да, нас устроит логика None в поле. Основная проблема была в том, что невозможно создать пользователя с ролью "Представитель команды". Попробуй пожалуйста у себя, т.е. у нас создаётся команда администратором, после мы должнать создать пользователя "представитель команды" И уже на этапе создания его привязывем к команде.

@Toksi86
Copy link
Contributor

Toksi86 commented Oct 10, 2024

Да, у меня сейчас всё именно так и работает, чуть позже сделаю пуш и создам PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
2 participants