Nummerierung in Ausfahrten-PDFs und Übersichten #119

Merged
christian.merten merged 5 commits from MK/counters into main 1 year ago
Collaborator
  1. ich habe die TN-Listen der generierten PDFs im Bereich Ausfahrten um eine laufende Nummer ergänzt, um die Übersichtlichkeit zu erhöhen
  2. den inline tab Teilnehmer*innen im Bereich Ausfahrten habe ich um einen TN und JL-Counter ergänzt. Das hilft ungemein während der Planung, gerade wenn die TN-Liste länger ist. Das war bisschen gefummel, verstehe wenn da jemand im Sinne einer sauberen code base was dagegen hat xD
1. ich habe die TN-Listen der generierten PDFs im Bereich Ausfahrten um eine laufende Nummer ergänzt, um die Übersichtlichkeit zu erhöhen 2. den inline tab Teilnehmer\*innen im Bereich Ausfahrten habe ich um einen TN und JL-Counter ergänzt. Das hilft ungemein während der Planung, gerade wenn die TN-Liste länger ist. Das war bisschen gefummel, verstehe wenn da jemand im Sinne einer sauberen code base was dagegen hat xD
marius.klein requested review from christian.merten 1 year ago
christian.merten reviewed 1 year ago
@ -920,6 +920,31 @@ class MemberOnListInline(CommonAdminInlineMixin, GenericTabularInline):
TextField: {'widget': Textarea(attrs={'rows': 1, 'cols': 40})}
}
sortable_options = []
template = "admin/edit_inline/tabular_footer.html"
Owner

Den Dateinamen finde ich verwirrend, wie wäre es mit admin/members/freizeit/memberonlistinline.html?

Den Dateinamen finde ich verwirrend, wie wäre es mit `admin/members/freizeit/memberonlistinline.html`?
@ -923,0 +924,4 @@
def people_count(self, obj):
if isinstance(obj, Freizeit):
# Number of organizers who are also in the Memberlist
Owner

Die "Rechnung" im if branch würde ich auch in models.py verschieben.

Die "Rechnung" im `if` branch würde ich auch in `models.py` verschieben.
Poster
Collaborator

meinst du so?

if isinstance(obj, Freizeit):
	# Number of organizers who are also in the Memberlist
	organizer_count = obj.staff_on_memberlist_count

	# Total number of people in the Memberlist
	total_people = obj.head_count
meinst du so? ``` if isinstance(obj, Freizeit): # Number of organizers who are also in the Memberlist organizer_count = obj.staff_on_memberlist_count # Total number of people in the Memberlist total_people = obj.head_count ```
Owner

Ja genau. Dann kann man das einfacher unit testen.

Ja genau. Dann kann man das einfacher unit testen.
Poster
Collaborator

voila 🙃

voila 🙃
christian.merten approved these changes 1 year ago
christian.merten left a comment
Owner

Merci!

Merci!
christian.merten merged commit 678559fb4d into main 1 year ago
christian.merten deleted branch MK/counters 1 year ago

Reviewers

christian.merten approved these changes 1 year ago
The pull request has been merged as 678559fb4d.
Sign in to join this conversation.
No reviewers
No Milestone
No project
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#119
Loading…
There is no content yet.