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

VPlan ID muss nicht per Request erfragt werden #95

Open
strifel opened this issue Jul 1, 2020 · 2 comments
Open

VPlan ID muss nicht per Request erfragt werden #95

strifel opened this issue Jul 1, 2020 · 2 comments
Labels
einfach enhancement New feature or request needs-api-implementation This issue needs (bigger) changes to the API

Comments

@strifel
Copy link
Member

strifel commented Jul 1, 2020

Momentan machen wir eine Request an /vplans nur um herauszufinden welche ID.
Die können wir aber auch ganz einfach im Client generieren.
Die würde wahrscheinlich die richtigen Ergebnisse generieren, solange darauf aufgepasst wird, dass am Wochenende der Montag genommen wird.
Durch die Methode würden in den Ferien auch keine Vertretungspläne statt ggf. den alten angezeigt werden.

@strifel strifel added bug Something isn't working enhancement New feature or request important This issues should be implemented ASAP medium labels Jul 1, 2020
@strifel strifel added this to the v1.0.0 milestone Jul 1, 2020
@strifel strifel self-assigned this Jul 1, 2020
@strifel strifel removed this from the v1.0.0 milestone Jul 27, 2020
@strifel
Copy link
Member Author

strifel commented Jul 27, 2020

Wir können diese Feature nicht mehr testen vor dem geplanten Release der 1.0, da kein VPlan existiert.

Die PR sollte getestet und gemerged werden sobald möglich.

@strifel
Copy link
Member Author

strifel commented Aug 12, 2020

Getestet.
Leider hatte ich vergessen, dass wir die ID nicht das Datum brauchen.
Ich merge nun die PR, da die bereits etwas verbessert, werde aber den Issue nicht schließen und warte auf https://github.com/kagonlineteam/sym-api/issues/69

@strifel strifel added einfach needs-api-implementation This issue needs (bigger) changes to the API and removed bug Something isn't working important This issues should be implemented ASAP medium labels Aug 12, 2020
@strifel strifel removed their assignment Jul 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
einfach enhancement New feature or request needs-api-implementation This issue needs (bigger) changes to the API
Projects
None yet
Development

No branches or pull requests

1 participant