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

Rozdělit renderování obsahu, aplikaci, a seznam kurzů #494

Open
12 of 22 tasks
encukou opened this issue Nov 28, 2018 · 2 comments
Open
12 of 22 tasks

Rozdělit renderování obsahu, aplikaci, a seznam kurzů #494

encukou opened this issue Nov 28, 2018 · 2 comments
Assignees
Labels

Comments

@encukou
Copy link
Member

encukou commented Nov 28, 2018

Aktuálně naucse není v dobrém stavu. Všechno souvisí se vším, a udělat nějaké změny a vylepšení je složité až nereálné. Rozplést to je těžší než se zdá. Zároveň je to frustrující pro lidi, kteří by se chtěli zapojit.
V posledních měsících mě zavalila práce, tak se to moc nepohnulo. Teď na to mám dedikovaný čas. Budu sem postupně psát, jak se posouvám.
(Kód je v mojí větvi, ale rozhodně není ve stavu, kdy bych chtěl aby se na to někdo díval nebo to kritizoval.)

Toto je refaktorování. Výstup bude nezměněný, takže se bude líp dělat review a na nic důležitého nezapomenu. (Opravy chyb a UX přijdou pak.)

Co plánuju v této fázi:

  • Rozdělení do tří knihoven:
    • Renderování obsahu – naucse_render
    • Webová aplikace – naucse
    • Obsah – naucse.python.cz
  • naucse bude s naucse_render komunikovat přes verzované a validované JSON API (a HTML – žádný Markdown).
  • naucse bude poskytovat verzované a validované JSON API (nástrojům jako je odevzdávátko úkolů nebo zpětnovazebník).
  • Když se nepodaří vyrenderovat jednu stránku, spadne build celého kurzu. (Teď spadne úplně celý build. Později se použije starší verze toho jednoho kurzu, a přestanu požadovat přístup k všem repozitářům s obsahem.)

Co mě od toho aktuálně dělí:

Hard deadline je v únoru.

Tohle blokuje (nebo řeší) všechno, čemu se chci věnovat (kromě obsahu). Zejména:

@encukou
Copy link
Member Author

encukou commented Jan 16, 2019

@frenzymadness, explicitní varování pro tebe: PR #505 s sebou nese zhoršení aktuálního stavu. Když se nepodaří vyrenderovat jednu stránku, spadne úplně celý build. To v praxi znamená, že když někdo do svého forku nahraje špatná data, zablokuje aktualizace celého naucse.
Já a Miro to můžeme vyřešit (máme přístup ke všem forkům), ale pro tebe to může být nepříjemné, kdybychom oba nebyli online :(
Oprava má docela prioritu (hned po statických souborech), takže to doufám problémy nepřinese.

@frenzymadness
Copy link
Member

Děkuji za varování. Tak snad se nic nepokazí hned ze startu, kdy se kromě přípravy kurzu nic zas tak často nemění.

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

No branches or pull requests

2 participants