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

Kalendar skusok #80

Closed
TomiBelan opened this issue Oct 13, 2015 · 12 comments
Closed

Kalendar skusok #80

TomiBelan opened this issue Oct 13, 2015 · 12 comments

Comments

@TomiBelan
Copy link
Member

Chcelo by to kalendar skusok, podobne ako mal Fajr.

@japdlsd
Copy link
Member

japdlsd commented Jun 5, 2017

mozem to spravit ja? v ramci magisterskeho projektu? :)

@mrshu
Copy link
Contributor

mrshu commented Jun 5, 2017

@japdlsd Spravit podla mna urcite. Ci v ramci magisterskeho projektu je uz narocnejsia otazka.

Myslim, ze v prvom rade je na mieste nejaky "scoping", aby bola predstava, ake featury ma vlastne takyto kalendar skusok mat.

Z mojho uhla pohladu to aktualne znamena odpovedat na otazky podobne tymto:

  • Pouzije sa na tento kalendar nejaky standardny komponent alebo sa bude vytvarat nejaky vlastny?
  • Kolko mesiacov bude takyto kalendar ukazovat? (Fajr demo ukazuje az 8 tyzdnov)
  • Bude sa robit aj iCalendar integracia? (Myslim, ze to bola killer featura)
  • Bude kalendar read-only alebo sa z neho bude dat aj na nejake terminy zapisat?

Z odpovedi na tieto otazky sa potom bude dat lahko povedat, do coho sa ma v kratkodobom/dlhodobom horizonte zmysel pustit. Osobne by som si vedel napriklad predstavit, ze bude tento projekt rozdeleny na dve casti:

  1. iCalendar dump prihlasenych terminov hodnotenia (relativne jednoducha ale vysoko pozitivna featura)
  2. Samotny kalendarovy listink vo VOTRi (narocnejsia ale tiez vysoko pozitivna featura)

Dolezita otazka je, kto bude reivewovat pull request(y), ktory/ktore tato issue urcite vygeneruje. Najlepsie by bolo, keby to mohol spravit @TomiBelan, v horsom pripade mozem pomoct aj ja.

@japdlsd
Copy link
Member

japdlsd commented Jun 5, 2017

ahoj :)

suhlasim s rozdelenim (tiez mi pride iCal integracia ako lepsia vec na zaciatok). Navyse, taketo poradie dovoluje (docasne) neriesit otazky 1,2 a 4 :)

co sa tyka Kralovica, tak by mi to mal uznat (to si doriesim osobne).

@japdlsd japdlsd self-assigned this Jun 5, 2017
@TomiBelan
Copy link
Member Author

Plán znie OK a pull requesty kľudne zreviewujem. (Teda až na to že o iCal nič neviem...) Ak budem pomalý, treba ma silno a často pingať. :-) A ak čokoľvek nebude jasné či už ako funguje alebo ako spraviť, treba sa ma odvážne a skoro pýtať.

@japdlsd
Copy link
Member

japdlsd commented Jun 15, 2017

spravil som tlacitko na export do iCal: japdlsd@e443d70
Da sa to spustit, treba len doinstalovat balicek file-saver (npm install file-saver).

Zatial je to len proof of concept, chyba niekolko veci:

  • ci je termin_key unikatny? je :)
  • dizajn (zatial to je len klasicke tlacitko)
  • pridanie balicku file-saver do dependencies (ako sa to ma robit?)
  • chceme importovat vsetky skusky za rok / iba toto skuskove / len buduce skusky?
  • par rad o tom, ako sa da lepsie zistit cas skusky nez parsovanim objektu terminy

Vedeli by ste mi poradit s tymito vecami? :)

@TomiBelan
Copy link
Member Author

Dizajn: klasicke tlacidlo je pre mna OK, aj klasicka linka by bola.

file-saver: Hmm. :/ Trochu by som dufal ze taketo nieco bude nativne a netreba ziadnu kniznicu. Ale nevyzera. Su dve moznosti:

  • a) Naozaj pouzit file-saver. Z masochistickych dovodov nemame package.json, ale v buildstatic.sh vsetko stahujeme a kopirujeme na spravne miesto. Najdi ako sa instaluju react a react-dom, a rovnako (npm install a potom cp bla/bla.min.js static/libs/) pridaj file-saver.
  • b) Nepouzit file-saver. Negenerovat kalendar pri kliknuti, ale okamzite. Viditelny button bude v skutocnosti linka s href="data:text/calendar,blablabla" download="skusky.ics". Takto netreba ziadne Bloby.

Ktore skusky: Neviem. Ake maju vyhody a nevyhody? Co je pre iCS pouzivatelov najprirodzenejsie?

Parsovanie: termin_key je naozaj unique (alebo aspon dufame) a vid structures.py ze z coho sa rata. Datum a cas dostavame ako stringy priamo z AISu a bez parsovania sa nezaobideme. Len ma napada, ze miesto split(".").reverse().join("") by som mozno spravil: var [den, mesiac, rok] = blabla.split(".") a potom var dtstart = rok + mesiac + den + "T" + hodina + minuta + "00", alebo dokonca var dtstart = `${rok}${mesiac}${den}T${hodina}${minuta}00`.

@mrshu
Copy link
Contributor

mrshu commented Jun 16, 2017

Re skusky: pre mna ako iCS pouzivatela maju uplne najvacsi zmysel nasledujuce skusky, na ktore som aktualne prihlaseny, kedze tie chcem urcite do svojho kalendara importovat.

V inych pripadoch prilis nevidim potrebu iCS exportu pre bezneho pouzivatela.

@peter-vasut
Copy link
Contributor

Môžem spraviť vizuálny kalendár ako Magisterský projekt? Rozprával som o tom s @mrshu . Plán (zostavený s pomocou Mareka) by bol nasledovný:

  1. pozrieť sa na Big Calendar:
    http://intljusticemission.github.io/react-big-calendar/examples/index.html
  2. Analyza moznosti pouzitia
  3. Navrh UI
  4. Skontrolovat slovencinu
  5. Farbicky carbicky (aby to bolo pekné červené)
  6. Pull Request (bolo by potom fajn ak by niekto, napríklad @TomiBelan spravil review)

@japdlsd japdlsd removed their assignment May 15, 2018
@TomiBelan
Copy link
Member Author

Urcite ano, potesim sa :-)

Tie Big Calendar priklady vyzeraju pekne. Ale ak nema samostatny js file, integracia do Votru mozno bude boliet. Stuchaj do @kubik369 nech dorobi #122. ;-)

Pull request by som nenechaval celkom na "krok 6". Kludne vzdy ked napises nejaky kus kodu, aj keby bol este nedokonceny alebo nefungoval, pushni to na nejaku vetvu (nie master;) a budem mat lepsi prehlad ako sa ti dari.

@peter-vasut
Copy link
Contributor

Nejako som nestíhal, ale plánujem sa do toho od zajtra po skúške pustiť. Ak chápem správne, branch si môžem vytvoriť priamo v tomto repozitári, nemusím robiť clone.

@TomiBelan
Copy link
Member Author

TomiBelan commented Jun 13, 2018 via email

@peter-vasut peter-vasut self-assigned this Jun 15, 2018
@peter-vasut
Copy link
Contributor

Pridal som novú vetvu kalendar_iss80. Od hotového to má zatiaľ ďaleko.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

4 participants