feat(members/admin): add training overview
#174
Merged
christian.merten
merged 57 commits from MK/training_tab into main 2 months ago
Reviewers
Request review
No reviewers
Labels
Clear labels
Betrifft die Administration des Kompass
Betrifft Anwender:innen des Kompass, das heißt nicht-Admins
Betrifft die Entwicklungs- und Produktiv Infrastruktur
Bedarf keiner strukturellen Änderungen und ist leicht umsetzbar, weil es ähnliche Funktionen schon gibt
Breaking change that won't be backward compatible
Something is not working
Documentation changes
Improve existing functionality
New functionality
This is security issue
Issue or pull request related to testing
Betrifft die Kompass Produktivumgebung von Heidelberg
Betrifft die Kompass Produktivumgebung von Ludwigsburg
The priority is critical
The priority is high
The priority is low
The priority is medium
Issue has been confirmed
This issue or pull request already exists
Invalid issue
This issue won't be fixed
Somebody has started to work on this but abandoned work
The pull request or issue awaits a response by the author to questions or comments by reviewers.
Something is blocking this issue or pull request
Feedback is required to reproduce issue or to continue work
Apply labels
Effekt
Admins
Betrifft die Administration des Kompass
Effekt
Anwender
Betrifft Anwender:innen des Kompass, das heißt nicht-Admins
Effekt
Infrastruktur
Betrifft die Entwicklungs- und Produktiv Infrastruktur
Issue gut geeignet für Einsteiger:innen
Bedarf keiner strukturellen Änderungen und ist leicht umsetzbar, weil es ähnliche Funktionen schon gibt
Kind/Breaking
Breaking change that won't be backward compatible
Kind/Bug
Something is not working
Kind/Documentation
Documentation changes
Kind/Enhancement
Improve existing functionality
Kind/Feature
New functionality
Kind/Security
This is security issue
Kind/Testing
Issue or pull request related to testing
Lokal
Heidelberg
Betrifft die Kompass Produktivumgebung von Heidelberg
Lokal
Ludwigsburg
Betrifft die Kompass Produktivumgebung von Ludwigsburg
Priority
Critical
The priority is critical
Priority
High
The priority is high
Priority
Low
The priority is low
Priority
Medium
The priority is medium
Reviewed
Confirmed
Issue has been confirmed
Reviewed
Duplicate
This issue or pull request already exists
Reviewed
Invalid
Invalid issue
Reviewed
Won't Fix
This issue won't be fixed
Status
Abandoned
Somebody has started to work on this but abandoned work
Status
Awaiting author
The pull request or issue awaits a response by the author to questions or comments by reviewers.
Status
Blocked
Something is blocking this issue or pull request
Status
Need More Info
Feedback is required to reproduce issue or to continue work
No Label
Effekt
Admins
Effekt
Anwender
Effekt
Infrastruktur
Issue gut geeignet für Einsteiger:innen
Kind/Breaking
Kind/Bug
Kind/Documentation
Kind/Enhancement
Kind/Feature
Kind/Security
Kind/Testing
Lokal
Heidelberg
Lokal
Ludwigsburg
Priority
Critical
Priority
High
Priority
Low
Priority
Medium
Reviewed
Confirmed
Reviewed
Duplicate
Reviewed
Invalid
Reviewed
Won't Fix
Status
Abandoned
Status
Awaiting author
Status
Blocked
Status
Need More Info
Milestone
Set milestone
Clear milestone
No items
No Milestone
Projects
Set Project
Clear projects
No project
Assignees
Assign users
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.
No due date set.
Dependencies
No dependencies set.
Reference: digitales/kompass#174
Reference in New Issue
There is no content yet.
Delete Branch 'MK/training_tab'
Deleting a branch is permanent. It CANNOT be undone. Continue?
No
Yes
Ich hab mich mal an einer neuen View für Fortbildungen versucht.
Navigationsgruppe "Ausbildung" enthält jetzt die Modelle Fortbildungen und Fortbildungstypen
permissions um die View zu sehen, ist aktuell view_trainings, da bin ich mir aber nicht sicher
Neues Feld "Activity" ermöglicht Angabe einer alpinen Spielart für jede Fortbildung
Tab hat eine Beschreibung, die Aufgabe der User klar macht.
Felder teilgenommen und bestanden sind sensibel, dafür habe ich analog Ausfahrten-Genehmigung eine permission angelegt, durch die nur eine Ausbildungsrolle eine bestandene GA/AM bestätigen kann (anhand hochgeladener TN-Bestätigung)
auch hier bin ich bei der permission nicht sicher, ob die so passt
Sieht gut aus. Ein paar Ideen:
Für ein follow up:
follow up:
edit:
allgemeingültige Benachrichtigung bräuchte ein create_date field auf allen relevanten Modellen... das wäre wohl ein Feature für sich...
WIP: MK/training_tabto MK/training_tab 4 months ago@ -12,12 +12,15 @@ JET_SIDE_MENU_ITEMS = [{'name': 'membernotelist', 'permissions': ['members.view_membernotelist']},{'name': 'klettertreff', 'permissions': ['members.view_klettertreff']},{'name': 'activitycategory', 'permissions': ['members.view_activitycategory']},Kannst du den
activitycategoryAbschnitt auch nach unten verschieben?@ -0,0 +3,4 @@from django.db import migrations, modelsclass Migration(migrations.Migration):Kannst du die migrations noch squashen?
@ -2062,1 +2063,4 @@'image/gif'])def __str__(self):return self.title + ' ' + self.date.strftime('%d.%m.%Y')datekannNoneseinich füge alternativ (ohne Datum) an.
@ -109,3 +109,3 @@<br><div class="app-members module current-app"><h2>Sonstiges</h2><h2>Ausbildung und Aktivtäten</h2>Typo: "Aktivitäten"
@ -0,0 +1,27 @@# Generated by Django 4.2.20 on 2025-10-10 15:50Kannst du das File in
0044_...umbenennen? Es gibt mittlerweile schon ein0043_....@ -474,15 +497,6 @@ msgstr ""msgid "Finance overview"msgstr "Kostenübersicht"#: members/admin.pyIch glaube das file musst du nochmal neu generieren. Das wird einen merge conflict geben.
@ -1406,0 +1417,4 @@class MemberTrainingAdmin(CommonAdminMixin, nested_admin.NestedModelAdmin):Kannst du die leere zeile entfernen?
@ -114,2 +115,4 @@ordering = ("date",)extra = 1field_change_permissions = {Das führt dazu, dass wenn nicht Admins eine Fortbildung eintragen, ein "Integrity Error" geworfen wird, weil
participatednicht null sein darf.Ich denke am Besten ist es einfach im
MemberTrainingmodel fürparticipatedundpassednull zu erlauben.Könntest du bitte auch noch ein
verbose_namefield immemberfield unterMemberTraininghinzufügen? Dann sieht es schicker aus in deiner neuen schicken Ansicht.@ -117,3 +123,4 @@class EmergencyContactInline(CommonAdminInlineMixin, admin.TabularInline):model = EmergencyContactdescription = _('Please enter at least one emergency contact with contact details here. These are necessary for crisis intervention during trips.')das gehört hier eigentlich nicht rein, soll ich das noch rausnehmen und wann anders mal in ne merge request machen?
Ja habe es gesehen. Wäre natürlich noch besser es separat zu machen, aber ich glaube es ist okay so.
Sieht alles sehr gut aus jetzt. Könntest du noch einen Test hinzufügen (in
members/tests/basic.pyin einer neuen KlasseTrainingAdminTestCase). Ich vermute es genügt einfach beide Seiten (changelistundchange) einmal aufzurufen, dann sind wir wieder bei 100% coverage.Sieht top aus jetzt!
MK/training_tabto feat(members/admin): add training overview 2 months agoad33d5db17into main 2 months agoReviewers
ad33d5db17.