MK/meeting_checklist #154

Merged
christian.merten merged 21 commits from MK/meeting_checklist into main 4 months ago
Collaborator

Dieses Feature ermöglicht die Generierung einer TN-Liste für alle Gruppen aus dem "Gruppen"-Tab. Dadurch werden langfristige Pain-points

  • Dokumentation der Anwesenheit der anvertrauten Kindern und Jugendlichen
  • Effiziente Anmeldung an der Kasse

gelöst.

Ich weiß noch nicht, ob die Angabe der Mitgliedsnummern der Kasse hilft. Konzept wird jetzt erstmal mit einzelnen Gruppen evaluiert.

Dieses Feature ermöglicht die Generierung einer TN-Liste für alle Gruppen aus dem "Gruppen"-Tab. Dadurch werden langfristige Pain-points - Dokumentation der Anwesenheit der anvertrauten Kindern und Jugendlichen - Effiziente Anmeldung an der Kasse gelöst. Ich weiß noch nicht, ob die Angabe der Mitgliedsnummern der Kasse hilft. Konzept wird jetzt erstmal mit einzelnen Gruppen evaluiert.
christian.merten reviewed 8 months ago
@ -0,0 +1,98 @@
{% load static common tex_extras %}

Kannst du hier das tex_base.tex template benutzen?

Kannst du hier das `tex_base.tex` template benutzen?
Poster
Collaborator

hatte ich versucht, brauche aber für die Tabelle ein anderes Layout (deutlich engere page margins, kein Briefkopf rechts), daher passt das Template nicht ganz. Ich könnte aber ein zweites base-layout dafür anlegen (jetzt oder sobald zweiter use case dazu kommt).

hatte ich versucht, brauche aber für die Tabelle ein anderes Layout (deutlich engere page margins, kein Briefkopf rechts), daher passt das Template nicht ganz. Ich könnte aber ein zweites base-layout dafür anlegen (jetzt oder sobald zweiter use case dazu kommt).

Du kannst den extra-preamble block benutzen, um mit \geometry{ ... } die margins zu ändern, also zB.

{% block extra-preamble %}
\geometry{margin=0mm}
{% endblock %}

Mit

{% block contact %}{% endblock %}

entfernst du den Briefkopf rechts.

Du kannst den `extra-preamble` block benutzen, um mit `\geometry{ ... }` die margins zu ändern, also zB. ``` {% block extra-preamble %} \geometry{margin=0mm} {% endblock %} ``` Mit ``` {% block contact %}{% endblock %} ``` entfernst du den Briefkopf rechts.
marius.klein marked this conversation as resolved

Soll die Anwesenheit auch dokumentiert bleiben? Also werden diese Zettel aufbewahrt oder digitalisiert?

Soll die Anwesenheit auch dokumentiert bleiben? Also werden diese Zettel aufbewahrt oder digitalisiert?
Poster
Collaborator

Soll die Anwesenheit auch dokumentiert bleiben? Also werden diese Zettel aufbewahrt oder digitalisiert?

aktuell ist aufheben, aber nicht digitalisieren geplant

> Soll die Anwesenheit auch dokumentiert bleiben? Also werden diese Zettel aufbewahrt oder digitalisiert? aktuell ist aufheben, aber nicht digitalisieren geplant

Soll die Anwesenheit auch dokumentiert bleiben? Also werden diese Zettel aufbewahrt oder digitalisiert?

aktuell ist aufheben, aber nicht digitalisieren geplant

Für die Kasse muss es analog als Papier vorliegen?

Denkbar wäre vllt auch ein QR code check in. Nach erfolgtem Check-In könnten die JLs der Gruppe die Liste per Mail an die Kasse schicken lassen (zB per Scan des selben QR codes durch Eingabe eines Gruppenpassworts oder per Scan eines (geheimen) anderen Abschick QR codes). Die gesammelten Check In daten könnten dann im "Klettertreff" model gespeichert werden.

> > Soll die Anwesenheit auch dokumentiert bleiben? Also werden diese Zettel aufbewahrt oder digitalisiert? > > aktuell ist aufheben, aber nicht digitalisieren geplant Für die Kasse muss es analog als Papier vorliegen? Denkbar wäre vllt auch ein QR code check in. Nach erfolgtem Check-In könnten die JLs der Gruppe die Liste per Mail an die Kasse schicken lassen (zB per Scan des selben QR codes durch Eingabe eines Gruppenpassworts oder per Scan eines (geheimen) anderen Abschick QR codes). Die gesammelten Check In daten könnten dann im "Klettertreff" model gespeichert werden.
Poster
Collaborator

Soll die Anwesenheit auch dokumentiert bleiben? Also werden diese Zettel aufbewahrt oder digitalisiert?

aktuell ist aufheben, aber nicht digitalisieren geplant

Für die Kasse muss es analog als Papier vorliegen?

Denkbar wäre vllt auch ein QR code check in. Nach erfolgtem Check-In könnten die JLs der Gruppe die Liste per Mail an die Kasse schicken lassen (zB per Scan des selben QR codes durch Eingabe eines Gruppenpassworts oder per Scan eines (geheimen) anderen Abschick QR codes). Die gesammelten Check In daten könnten dann im "Klettertreff" model gespeichert werden.

Ich stimme zu, das wäre eine cleane Lösung, scheitert aber meines Wissens aktuell an der mangelnden digitalen Infrastruktur der Kasse :(

> > > Soll die Anwesenheit auch dokumentiert bleiben? Also werden diese Zettel aufbewahrt oder digitalisiert? > > > > aktuell ist aufheben, aber nicht digitalisieren geplant > > Für die Kasse muss es analog als Papier vorliegen? > > Denkbar wäre vllt auch ein QR code check in. Nach erfolgtem Check-In könnten die JLs der Gruppe die Liste per Mail an die Kasse schicken lassen (zB per Scan des selben QR codes durch Eingabe eines Gruppenpassworts oder per Scan eines (geheimen) anderen Abschick QR codes). Die gesammelten Check In daten könnten dann im "Klettertreff" model gespeichert werden. Ich stimme zu, das wäre eine cleane Lösung, scheitert aber meines Wissens aktuell an der mangelnden digitalen Infrastruktur der Kasse :(
marius.klein changed title from WIP: MK/meeting_checklist to MK/meeting_checklist 5 months ago
Poster
Collaborator

@christian.merten ich bin leider nicht drum gekommen, ein neues member field anzulegen. 🙈 Was hältst du von dem Ansatz? Ich würde kommende Woche den finalen Test mit der Hallenkasse machen, ansonsten wäre das Feature fertig.

@christian.merten ich bin leider nicht drum gekommen, ein neues member field anzulegen. 🙈 Was hältst du von dem Ansatz? Ich würde kommende Woche den finalen Test mit der Hallenkasse machen, ansonsten wäre das Feature fertig.
marius.klein requested review from christian.merten 5 months ago

Was genau ist die ticket_no? Ist das die Nummer der Dauerkarte? Braucht die Kasse das zwingend?

Was genau ist die `ticket_no`? Ist das die Nummer der Dauerkarte? Braucht die Kasse das zwingend?
christian.merten reviewed 5 months ago
@ -869,0 +892,4 @@
}
return render_tex(f"Gruppen-Checkliste", 'members/group_checklist.tex', context)
def mondays_until_nth(n):

Das sollte wahrscheinlich in utils.py.

Das sollte wahrscheinlich in `utils.py`.
marius.klein marked this conversation as resolved
@ -368,0 +375,4 @@
@property
def ticket_tag(self):
"""Returning the ticket number stripped of strings and spaces"""

- newline

`- newline`
marius.klein marked this conversation as resolved
christian.merten refused to review 5 months ago
Poster
Collaborator

Was genau ist die ticket_no? Ist das die Nummer der Dauerkarte? Braucht die Kasse das zwingend?

genau, das ist eine Eintrittskarten-Nummer, die nicht der Mitgliedsnummer entspricht 🤦‍♂️ wobei das wahrscheinlich gar nicht so selten ist.

> Was genau ist die `ticket_no`? Ist das die Nummer der Dauerkarte? Braucht die Kasse das zwingend? genau, das ist eine Eintrittskarten-Nummer, die nicht der Mitgliedsnummer entspricht 🤦‍♂️ wobei das wahrscheinlich gar nicht so selten ist.

Was genau ist die ticket_no? Ist das die Nummer der Dauerkarte? Braucht die Kasse das zwingend?

genau, das ist eine Eintrittskarten-Nummer, die nicht der Mitgliedsnummer entspricht 🤦‍♂️ wobei das wahrscheinlich gar nicht so selten ist.

Okay, ist nicht ideal, weil man zur Zeit die angezeigten Felder nicht konfigurieren kann.
Es wäre schick, wenn in der settings.toml manche Felder ausgeblendet werden könnten.

Aber das Problem besteht schon länger, wir können es also jetzt mal so machen. Kannst du noch einen Kommentar in members/models.py hinzufügen, der ticket_no erklärt?

Und ist ticket_no an der richtigen Stelle? Vielleicht wäre es (evtl. gemeinsam mit der DAV Mitgliedsnummer) unter "Sonstiges" gut aufgehoben?

> > Was genau ist die `ticket_no`? Ist das die Nummer der Dauerkarte? Braucht die Kasse das zwingend? > > genau, das ist eine Eintrittskarten-Nummer, die nicht der Mitgliedsnummer entspricht 🤦‍♂️ wobei das wahrscheinlich gar nicht so selten ist. Okay, ist nicht ideal, weil man zur Zeit die angezeigten Felder nicht konfigurieren kann. Es wäre schick, wenn in der `settings.toml` manche Felder ausgeblendet werden könnten. Aber das Problem besteht schon länger, wir können es also jetzt mal so machen. Kannst du noch einen Kommentar in `members/models.py` hinzufügen, der `ticket_no` erklärt? Und ist `ticket_no` an der richtigen Stelle? Vielleicht wäre es (evtl. gemeinsam mit der DAV Mitgliedsnummer) unter "Sonstiges" gut aufgehoben?
Poster
Collaborator

Danke!

ich habe die zwei Anpassungen gemacht.

Danke! ich habe die zwei Anpassungen gemacht.

Top, du sagtest ihr wolltet das nochmal final testen? Oder ist das schon passiert?

Top, du sagtest ihr wolltet das nochmal final testen? Oder ist das schon passiert?
Poster
Collaborator

das ist gestern passiert. Insofern ist es von meiner Seite aus ready.

das ist gestern passiert. Insofern ist es von meiner Seite aus ready.

Der pull request hat noch merge conflicts. Kannst du die fixen?

Der pull request hat noch merge conflicts. Kannst du die fixen?
Poster
Collaborator

Danke für den Hinweis, jetzt sollte es passen

Danke für den Hinweis, jetzt sollte es passen

Ich bekomme eine Fehlermeldung "name 'mondays_until_nth' is not defined" (fehlt vmtl der Import).

Edit: hab es gefixed.

Ich bekomme eine Fehlermeldung "name 'mondays_until_nth' is not defined" (fehlt vmtl der Import). Edit: hab es gefixed.
christian.merten reviewed 4 months ago
@ -0,0 +21,4 @@
% HEADLINE
{\noindent\Large{Gruppenliste {{ group.name }} }}\\[1mm]
\noindent {{ weekdays|index:group.weekday|esc_all }}, {{ group.start_time }} - {{ group.end_time }} Uhr\\

Hier wird bei fehlenden Infos None ausgespuckt. Können wir da anstatt dessen --- eintragen? Oder die Zeile weglassen?

Hier wird bei fehlenden Infos `None` ausgespuckt. Können wir da anstatt dessen `---` eintragen? Oder die Zeile weglassen?
Poster
Collaborator

ich gebe jetzt einfach eine leere Zeile aus, wenn die Infos fehlen.

ich gebe jetzt einfach eine leere Zeile aus, wenn die Infos fehlen.
marius.klein marked this conversation as resolved
Poster
Collaborator

ich hab jetzt den Branch auf testing nicht zum laufen bekommen (verlässt den Wartungsmodus nicht). Das mit den migrations war schon davor mal ein Problem auf testing, ich kann aber nicht so richtig einsehen, woran das genau liegt.

ich hab jetzt den Branch auf testing nicht zum laufen bekommen (verlässt den Wartungsmodus nicht). Das mit den migrations war schon davor mal ein Problem auf testing, ich kann aber nicht so richtig einsehen, woran das genau liegt.
christian.merten merged commit a75208b41c into main 4 months ago
The pull request has been merged as a75208b41c.
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#154
Loading…
There is no content yet.