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.

Sign up for GitHub

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

Jump to bottom

COURSES: Kursfilter "Nur mit verfügbaren Plätzen" returned 500er #1494

Closed
njaeggi opened this issue Jan 9, 2025 · 2 comments · Fixed by hitobito/hitobito#3123
Closed

COURSES: Kursfilter "Nur mit verfügbaren Plätzen" returned 500er #1494

njaeggi opened this issue Jan 9, 2025 · 2 comments · Fixed by hitobito/hitobito#3123
Assignees
@amaierhofer
Labels
bug Something isn't working interest-sac

Comments

@njaeggi
Copy link
Contributor

njaeggi commented Jan 9, 2025

Aktuelles Verhalten

Sentry Issue: https://sentry.puzzle.ch/pitc/hitobito-backend/issues/73943

Wenn man im Kursfilter nach Kursen mit verfügbaren Plätzen filtert bekommt man aktuell eine Fehlermeldung.

Gewünschtes Verhalten

Kursfilter funktioniert ohne 500er Error

Schritte zu reproduzieren

  1. /list_courses aufrufen
  2. Unter "Anmeldung" die Filteroption "Nur mit verfügbaren Plätzen" auswählen

Wann ist der Fehler aufgetreten?

09.01.2025

hitobito version

2.3.12

Organisation

SAC

Screenshots, weiterer Kontext

No response

ToDos

No response
@njaeggi njaeggi added bug Something isn't working needs-confirmation labels Jan 9, 2025
@njaeggi njaeggi self-assigned this Jan 9, 2025
@njaeggi njaeggi linked a pull request Jan 9, 2025 that will close this issue
Use table names in places available query hitobito/hitobito#3123
Merged
@njaeggi njaeggi removed their assignment Jan 10, 2025
@amaierhofer
Copy link
Contributor

Auf der PBS INT tritt das nicht auf, auch bei anderen hab ich es nicht gesehen. Was genau läuft da bei SAC anders?

@njaeggi
Copy link
Contributor Author

njaeggi commented Jan 20, 2025

Bei SAC wird ein Attribut zu Event::Kind hinzugefügt (maximum_participants), mit dem genau gleichen Namen wie das Attribut auf den Events, da wir beide im Query selecten, ist diese Column ambigious.

Und ganz im allgemeinen macht es viel Sinn die maximum_participatns im core mit dem table name zu selektieren, da auch die anderen Columns im Filter Query alle mit "events." selektiert werden.

@njaeggi njaeggi assigned amaierhofer and unassigned njaeggi Jan 20, 2025
njaeggi added a commit to hitobito/hitobito that referenced this issue Jan 22, 2025
@njaeggi
Use table names in places available query (hitobito/hitobito_sac_cas#…
57aad27 
…1494)

* Use table names in available places available

* Adjust specs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@amaierhofer amaierhofer

Labels
bug Something isn't working interest-sac
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Use table names in places available query hitobito/hitobito
2 participants
@amaierhofer @njaeggi