-
Notifications
You must be signed in to change notification settings - Fork 0
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
Matush/Semester picker in administration #323
Conversation
e7d8eaf
to
1c79191
Compare
1c79191
to
e418b01
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Preklikala som si to, je to funkcne a vyzera to v pohode. Kod som nepozerala nejako podrobne, aj tak sa necitim kvalifikovana sa k nemu vyjadrit.
Ja som to povodne nedavala do topgridu, ale tam nizsie pre to, ze mi prislo, ze tam v topgride by som si to nevsimla, keby prvykrat to mam pustene. Na druhu stranu takto sa to pouziva vo vysledkoch aj zadaniach, takze to je aspon jednotne a vlastne to dava vacsi zmysel. Takze si myslim, ze to moze byt takto.
Na razcestnik sa kludne moze zalozit nova issue, ale myslim ze to nie je take podstatne, takze podla mna kludne do 2.0. Takto ako to teraz je mi to pride dostatocne funkcne.
uz som ho zalozil a nahodil do 2.0 #324 |
build on #316, close #303 , ale mozno by sme chceli zalozit nove issue na vytvorenie novej page, ktora by osahovala nejake rascestnik do buducnosti
Trochu som sa s tym hral, avsak nie som si isty ze ci to posunut do topgridu je dobry napad.@vikibrezinova pozri sa na to prosim a povedz ci to takto moze bytPrerobil som url pre seminar administraciu aby zobrazovala
/{rocnik}/{semester}
takze semester picker vie detekovat z URL co je zvoleny semseter.Nasledne bolo este potrebne:
Spat na semester
v administracii ulohy