Skip to content
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

Add students list for study courses, study classes & courses #712 #713 #750

Merged
merged 3 commits into from
Dec 12, 2024

Conversation

hupf
Copy link
Collaborator

@hupf hupf commented Dec 2, 2024

Siehe #712 #713

Allgemein:

  • Route, Komponenten & Grundlage für Data Fetching
  • Suche
  • Backlink zu Ursprungsliste
  • Tests schreiben
  • Umstellung von /Students auf /Persons
  • Mehrfachauswahl (Checkboxen) sind nicht möglich (zurückbauen)
  • Auf «Aktuelle Fächer»: Count bei Studiengang nicht über Event.StudentCount sondern die Subscriptions zählen.
  • Squashen

Fach- & Klassenliste:

  • Data Fetching
  • Basic Layout
  • Link auf Dossiers mit Backlink
  • Anzeige Klasse(n) im Titel
  • Anzeige Klasse(n) bei Student, falls mehr als eine pro Fach (Workaround über LessonPresences)
  • Anzeige Lehrbetrieb
  • Reports
  • E-Mail an alle

Studengangsliste:

  • Data Fetching
  • Basic Layout
  • Sortierung
  • Anzeige Status
  • Anzeige Anmeldedatum
  • Mehrfachauswahl

@hupf hupf assigned hupf and caebr Dec 2, 2024
@hupf hupf force-pushed the feature/712-713-event-details branch from ca99f08 to 8dedf3b Compare December 2, 2024 13:43
@caebr caebr force-pushed the feature/712-713-event-details branch 2 times, most recently from 912b34e to 39eeb99 Compare December 4, 2024 16:25
@hupf hupf force-pushed the feature/712-713-event-details branch 2 times, most recently from 79cb589 to cbe97aa Compare December 5, 2024 09:51
@hupf
Copy link
Collaborator Author

hupf commented Dec 9, 2024

@caebr Die Nachbesserungen sind gemacht, willst du noch einmal darüberschauen?

  • Umstellung auf /Persons Endpunkt, damit wir die richtigen Daten haben: 9dffc3b
  • Ausbau von Checkboxes, Edit Button und Selection Service, da es momentan kein Edit gibt: 016af68
  • Zählen der Studenten für Studiengänge auf der Liste «Aktuelle Fächer» via Subscriptions, damit die Anzahl mit «X Anmeldungen» stimmt: b9194f2

Falls alles i.O. werde ich die ersten beiden Commits dann noch squashen und danach mergen. Der dritte Commit würde ich separat lassen, da es ein Fix auf der bestehenden «Aktuelle Fächer» Liste ist.

@hupf hupf changed the title Draft: Add students list for study courses, study classes & courses #712 #713 Add students list for study courses, study classes & courses #712 #713 Dec 11, 2024
@hupf hupf mentioned this pull request Dec 11, 2024
5 tasks
@hupf hupf force-pushed the feature/712-713-event-details branch from b9194f2 to fa3f24f Compare December 12, 2024 09:13
@hupf hupf merged commit 9d4dce7 into main Dec 12, 2024
2 checks passed
@hupf hupf deleted the feature/712-713-event-details branch December 12, 2024 09:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants