PSE-WS1617-IPD-Workflow / PSE_WS1617_IPD_Boehm_Studienplan

Other
4 stars 0 forks source link

Client still has min-semester-ects slider etc. but server ignores it #217

Closed algorhythmus closed 7 years ago

algorhythmus commented 7 years ago

Currently, the server ignores all parameters except the preferred subjects and max-semester-ects. min-semester-ects and min-/max-semesters are ignored. Maybe we should remove the corresponding GUI elements?

13hannes11 commented 7 years ago

Wie wir hiermit verfahren würde ich von der Generierung abhängig machen.

13hannes11 commented 7 years ago

image Hab ich beim REST-Teil der Entwurfsdokumentation entdeckt...

algorhythmus commented 7 years ago

Wird aber im Implementierungsbericht auf S. 9 berichtigt und da steht nur max-semester-ects. (Die Generierung benutzt (momentan) auch nur genau das.)

13hannes11 commented 7 years ago

Wird aber im Implementierungsbericht auf S. 9 berichtigt und da steht nur max-semester-ects. (Die Generierung benutzt (momentan) auch nur genau das.)

Das erklärt die Diskrepanz.

Also Rausschmeißen werden wir es beim Client erst am Ende, gibt genug wichtigeres, sollte beim Server genügend Zeit sein, kann man ja Versuchen das noch zu implementieren (erst die Bugs in der Generierung beheben).

algorhythmus commented 7 years ago

Server does support the parameters in question. Please reactivate the features!

samysweb commented 7 years ago

It would have been great, if someone would have informed us about that prior to the closing of this bug!