-
Notifications
You must be signed in to change notification settings - Fork 2
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
MIL-107 m2m связь пользователей, направлений и уровней #403
base: dev
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
нужно объединить миграции
добавить тесты для открытия форм
для решения конфликтов, проще всего сделать новую ветку и туда перенести изменения. |
"author", | ||
"updated_at", | ||
) | ||
list_display = ("title", "author", "created_at", "updated_at") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
оставь как было
"""Aдминка для Division.""" | ||
|
||
ordering = ("-updated_at",) | ||
list_display = ("title", "author", "created_at", "updated_at", "courses") | ||
list_display = ("title", "author", "updated_at", "courses") | ||
exclude = ("author",) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
автора нужно добавить в raw_field (посмотри по проект по "row")
list_display = ("id", "volunteer", "division", "level") | ||
list_display_links = ("volunteer",) | ||
list_filter = ( | ||
"volunteer", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
волонтера добавь в raw_field
@@ -113,3 +129,4 @@ class VolunteerAdmin(admin.ModelAdmin): | |||
admin.site.register(Department) | |||
admin.site.register(Badge, BadgeAdmin) | |||
admin.site.register(Level, LevelAdmin) | |||
admin.site.register(UserDivisionLevel, UserDivisionLevelAdmin) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
отдельно таблицу связей отображать не надо
в карточке пользователя добавь отображение списка направлений
"courses.Division", | ||
through="users.UserDivisionLevel", | ||
blank=True, | ||
related_name="volunteers", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
здесь указывается имя для обратной связи,
related_name="+",
No description provided.