tomas-dostal / potrebujurousku

A repo for site potrebujurousku.cz - site with simplified info about covid regulations in the Czech republic

Potrebujurousku.cz

Long time ago...

Back in the days when the COVID pandemic struck in 2020, there was a need for some sort of website that would help summarize the chaotic regulations from ministries. This project was alive for only a few months, and what you can see now is a tombstone of something I put a bit of my heart into. Now I see all the imperfections, and I would build it from scratch differently, but at least it was a very valuable lesson to me. I would like to express my gratitude to anyone who contributed in any way to this project. Thank you very much, and I am very happy that the whole pandemic thing is over now.

Below, you can find a bit of the backstory (CZ) of how everything happened and how things worked, etc.

• 
• GitHub repository of the report
• Report (PDF)

Here, you can find a few screenshots of how the site looked back then. Please keep in mind that none of the information in the screenshots or repository is up-to-date

Peace!

potrebujurousku's Issues

[UX] Matoucí buttony?

Problém

Ty buttony jsou pro mě matoucí - čekal bych, že když na ně kliknu (zobrazují se jako kliknutelné), tak to něco udělá - pravděpodobně vyfiltruje typ opatření.

Návrh řešení

Buď bych změnil, aby se na to nedalo kliknout (možná přes disabled, nebo z toho udělat span, který má ty classes?).
Nebo bych tam přidal to filtrování (nevím ale, jestli je to funkce, kterou by někdo chtěl využít)

Odstranění odhalených duplikátů odhalených při kontrole aktualizací

Z nějakého důvodu začali na mzdr umisťovat do WP příspěvků i spoustu stejných odkazů na PDFka. Při kontrole aktualizací se odhalí všechny, ale některé tam jsou vícekrát.

TODO: Místo listu možná použít množinu, nebo nějak kontrolovat, jestli se v listu už totožný prvek nevyskytuje

Dokumentace rozjetí projektu pro vývojáře

[UI] Zobrazení opatření podle kategorií v kartách

Bootstrapovina, chtělo by to něco takového

Konvence psaní kódu a práce s repozitářem

Přijde mi dobrý mít nějaký sdílený konvence psaní kódu a práce s repozitářem.

Já si hodně zvykl pro práci s kódem používat následující:

• na formátování black v kombinaci s flake8 linterem
  mám to napojené na pre-commit hooks, takže se mi to automaticky kontroluje/formátuje při commitu.
• pak mám ještě bandit na kontrolu bezpečnostních nedostatků.

V repozitáři jsem pak zvyklý na následující:

• nový kód přidávat přes pull requests, napojené na další automatické kontroly kódu skrz github actions (v public repo jsou zadarmo, tady by asi šlo mít jedině vlastní), a případně se systémem approvals od dalších vývojářů. Přijde mi dobré, aby merge prováděl ten, kdo vytvořil PR.
• do hlavní větve přidávat přímo maximálně nějaké hotfixy, opravy textace.

Taky by mě zajímalo, jestli preferujeme naming český nebo anglický. Teď je to takový mix, tak nevím co mám preferovat, když něco přepisuju.

Ale to jsou jenom moje zvyky, a návrhy rád bych tady vytvořil prostor pro diskuzi :)
A je to asi hlavně na @tomas-dostal jako šéfa :)

Indexace pro vyhledávače (vygenerovat sitemap.xml)

Přidání angličtiny

Django k tomu píše: https://docs.djangoproject.com/en/3.1/topics/i18n/

Migrace databáze, přepis do ORM

O co jde

Používání Oracle databáze není udržitelné, chceme se přesunout jinam.
Zároveň chceme zlepšit kvalitu kódu.

Otázky

Chceme MySQL nebo PostgreSQL?

• PostgreSQL je teď hodně populární, ale vlastně se moc neorientuju v jejích výhodách. Jedna, o které vím, je, že podporuje transakce u DDL queries (oproti MySQL).

Plán

1. Vytvořit lokální databázi na základě schématu té původní.
2. Napojit novou databázi na Django
3. Vygenerovat ORM modely
4. Vytvořit datové import migrace (pro konstantní data)
5. Vytvořit databázovou základní migraci (vytvoření tabulek)
6. Přepsat SQL dotazy do ORM variant.
7. Spustit na serveru

Základní testy

Testy jsou velká kapitola, a sám s nima nějak zvlášť neumím, ale přišlo by mi fajn napsat si nějaký úplně základní, které usnadní vývoj.

Pro začátek by mohlo stačit třeba nějaké, co kontrolují, že všechny stránky vrací kód 200, tedy nepadají na nějaké chybě.

Monitoring errorů

Chceme sledovat errory, zkusíme Sentry

Pushování do masteru

@tomas-dostal Ve FAQ píšeš, že se nemá pushovat do masteru. V nastavení repozitáře se to dá i úplně zakázat, tak bych byl pro, abys to udělal.

Možná tam jde i nastavit, abys třeba ty mohl, ale nejsem si tím jistý. Zároveň je to možná zbytečné - i když chceš udělat hotfix, tak se dá vytvořit PR a hned ho mergnout, není to o tolik víc práce :)