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

Verbesserung API - get/post Wahlen #441

Open
MrSebastian opened this issue Sep 25, 2024 · 0 comments
Open

Verbesserung API - get/post Wahlen #441

MrSebastian opened this issue Sep 25, 2024 · 0 comments
Labels
enhancement New feature or request service-basisdaten

Comments

@MrSebastian
Copy link
Member

MrSebastian commented Sep 25, 2024

Mit Erreichung von Milestone 1 sehen der Requests vie folgt aus:

  • GET /businessActions/wahlen/{wahltagID}
  • POST /businessActions/wahlen/{wahltagID}

Aus der URL für Get könnte man schließen das rein die wahltagID eine Rolle. Dies ist aber nicht der Fall. Anhand der WahltagID wird ein Wahltag ermittelt. Desssen Datum wird dann für das ermitteln der Wahlen verwendet.

Besser wäre aus meiner Sicht, sofern die Logik so bleiben soll, GET /businessActions/wahlen?byDatumOfWahltag={wahltagID}.

Dies spiegelt den filternden Charakter besser wieder. Ein Pathvariable in der bestehende URL sollte eine WahlID sein.

Aus dem POST sollte aus meiner Sicht ein PUT werden. Die Pathvariable kann auch entfernt werden.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request service-basisdaten
Projects
None yet
Development

No branches or pull requests

1 participant