Místa: hledání a další drobná vylepšení

Přijde mi dobré.

Při vyhledávání na stránkách míst je trochu zvláštní, že tam pak nad sebou jsou dvě tabulky, ale každá zobrazuje něco jiného a klidně mohu chtít použít původní tabulku, když mi vyhledávání nepomůže, takže bych ji neskrýval. Takže nevím co s tím, ale na první pohled mi to přišlo zvláštní

Souhlasím, dvě tabulky jsou zvláštní. Také by se mi asi více líbila jedna s tím, že při příchodu na stránky by se vyhledával prázdný řetězec = vše.

Ale to právě není totéž, druhá tabulka zobrazuje přímé potomky místa.

Jo, teď už mi to dává smysl. Díky za vysvětléní.

Přijde mi nejrozumnější tam obě tabulky nechat.

Přemýšlím, jestli na stránce http://localhost:5000/org/place/2/roles nechceme nějak schovat/oddělit/zašedit zděděné pole. Například je trošku nešikovné, když si otevřeš stránku nějaké školy a uvidíš tam smečku celostátních opravovatelů MO-P. Také mě vlastně není jasné, proč na to máme stránku nejedná se jen o odkaz na https://osmo.matematickaolympiada.cz/org/org/?is_role_filter=yes&search_name=&search_email=&search_year=&search_category=&search_seq=&search_in_place=BK&search_right_for_place=&submit=Filtrovat&offset=0&limit=50, kde už většina podobných problému je vyřešena.

Zděděné role zkusím oddělit do samostatné tabulky.

Souhlasím, že obecné hledátko orgů umí zobrazit totéž, ale tohle mi pro základní přehled přijde mnohem přehlednější.

No mě naopak připadá, že udržovat dvě instance toto stejného není dobrá cesta. Jednak je to víc práce, druhak to mate uživatele, protože mají pocit, že se jim funkce mění pod rukama. (Vždyť přece ten výpis organizátorů minule uměl i filtrovat, proč to teď neumí?)

Jenže ony to nejsou dvě instance téhož, spíš dva různé pohledy na stejná data. Otázka "kdo se stará o toto místo" mi přijde natolik častá, že stojí za to udržovat samostatný pohled, který to ukáže co nejpřehledněji. Podobně jako se hodí mít na hlavní stránce přehled soutěží.

Případnému zmatení uživatelů by mohlo jít bránit odkazem z této tabulky do obecného seznamu orgů.

kdo se stará o toto místo

Jo, to dává smysl, jen aktuální tabulka zobrazuje něco úplně jiného. Chtěná data totiž zapadnou mezi spoustu celostátních rolí (ideálně ještě k 5 let starým MO), kteří o ZŠ Horní Dolní nikdy neslišeli.

To by se mělo vyřešit tím rozdělením na dvě tabulky.

Jinak historické role pro specifické ročníky bych rád automaticky promazával.

Rozdělil jsem.

changed the description

added 3 commits

• 128451f8 - Místa: Výpis organizátorů odděluje přidělené role od zděděných
• 9ff032ef - Místa: Vyhledávání uvažuje každé slovo dotazu zvlášť
• 523b9f9b - Místa: Formulář pro hledání používá GET

Compare with previous version

Ještě jsem formulář předělal na GET. Ochrana proti CSRF tady není potřeba, formulář nemá žádné side-efekty. A je milé mít funkční tlačítko Zpět.

Merguji, dík za review!

merged

mentioned in commit 7070d8f6