search

ZdruzenieSTROM / webstrom-frontend

4 stars 0 forks source link

Semester picker in administration #316

Closed vikibrezinova closed 7 months ago

vikibrezinova commented 7 months ago

Nie je to velmi pekne spravene, lebo som len upravila semesterPicker, ktory sa pouziva vo vysledkoch a zadaniach, aby sa dal pouzit aj tu. Ale malo by to byt aspon funkcne.

Matushl commented 7 months ago

trochu sa to skroluje ponad - pokusil som sa spravit druhe PR kde som ten semester picker umiestil inde... ale nechcel som ti to pushovat do tohto PR, lebo neviem teraz vecer uz zhodnotit ci to je dobry napad image

Matushl commented 7 months ago

taktiez v semesterPickery sa nenastavia spravne data, pretoze to nevie z URL precitat ze o ktory semester sa jedna. Je totiz zvyknuty zo stranok zadani a rieseni ze v url je rocnik a nie ID semstra (to znamena ze by nefungovalo keby si nastavila title) - zacinam uvazovat ci je dobry napad pouzit ten isty semster picker (aj ked uprimne si myslim ze to je spravny pristup, lebo ten semesterPicker je dost pouzitelny, len treba zmenit nejake veci v administracii :-D )

Matushl commented 7 months ago

resenie bolo nakoniec mergnute v https://github.com/ZdruzenieSTROM/webstrom-frontend/pull/323