Nummerierung in Ausfahrten-PDFs und Übersichten #119

Merged
christian.merten merged 5 commits from MK/counters into main 11 months 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 11 months ago
christian.merten reviewed 11 months 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"

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

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 ```

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 11 months ago
christian.merten left a comment
Owner

Merci!

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

Reviewers

christian.merten approved these changes 11 months 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.