opened
veronikaMeciarova
username_0
<!-- Reviewable:start -->
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/fmfi-svt/votr/125)
<!-- Reviewable:end -->
Pridane vytvaranie zapisnych listov
issue
created
Adman
username_1
@username_2 Mozes to reviewnut prosim?
comment
created
TomiBelan
username_2
Dakujem za pull request. Este som to nestihol otestovat a spravit poriadny code review, ale moja prva otazka je: nie je idealne ak je v selectore stale linka ale v prehlade studia vacsinou nie je ziadne tlacidlo na pridanie zapisneho listu. Dalo by sa spravit, aby ta linka pouzila rovnaku podmienku a ukazovala sa iba ak je mozne skusit vytvorit zapisny list?
comment
created
mrshu
username_3
@username_0 ping
comment
created
TomiBelan
username_2
Vďaka za update! Večer alebo zajtra by som mal mať čas pozrieť si to.
comment
created
Adman
username_1
@username_2 aktualne je obdobie vytvarania zapisnych listov, tak keby to stihnes reviewnut, mohlo by to vela ludom pomoct :)
comment
created
TomiBelan
username_2
@username_1, ďakujem za pripomenutie - priznávam sa, zabudol som že som na ťahu, a potom som mal málo času. Konečne som späť. Snáď sa ešte nájde nejaký opozdilec, ktorý ešte nemá vyrobený zápisný list a ktorému sa to zíde.
@username_0, dobrá práca. Aby som to ďalej nezdržoval, nebudem písať ďalšie komentáre a proste spravím vhodné zmeny, aby som to mohol hneď mergnúť a nasadiť na votr.uniba.sk.

Pre záujem, zmeny sú v https://github.com/fmfi-svt/votr/pull/125/commits/b41b02782f027ea41f063290b98758bfb5523275 a sú to tieto:
* console.py vrátené na pôvodnú verziu, ktorá skúša aj local_settings aj default_settings
* v ZapisnyListSelector.js namiesto globálnej premennej getItems() vracia dvojicu
* áno ten confirm som myslel inak, chcel som povedať že pred tým ako zavoláme sendRpc by sme sa mohli opýtať či si je užívateľ istý (a či máme správny akademický rok). Na ukazovanie chýb naozaj stačí alert.
* pridal som `if (!cache.loadedAll) { return <Loading requests={cache.missing} />; }` - zhodou okolností to fungovalo aj bez toho lebo PrehladStudiaZapisneListy robí rovnaký request
* `class=` -> `className=` (myslím že class= v Reacte spôsobí warning) a `return ''` -> `return null` (to je len drobnosť ale trochu optimálnejšie)
* drobnosť: zmazané medzery na konci riadkov
comment
created
TomiBelan
username_2
Už to beží na https://votr.uniba.sk. Niekto mi prosím dajte vedieť, či všetko funguje :)
(Snáď som so svojimi poslednými zmenami nič nepokazil.)

Gratulujem, magisterský projekt je úspešne ukončený :)
comment
created
Adman
username_1
@username_2 @username_0 Ahojte. Chcel som sa k tomuto spytat, ci ten button je aj na "Zapis predmetov" podstranke. Ak nie, mozno by bola dobra feature ho tam pridat, kedze na stranku "Prehlad studia" ludia moc nechodia.
comment
created
TomiBelan
username_2
Áno, je.
comment