ARUP-CAS / aiscr-webamcr

Archaeological Map of the Czech Republic (AMCR)
https://amcr-info.aiscr.cz/
GNU General Public License v3.0
5 stars 0 forks source link

Úprava původních klíčů pro překlady #558

Closed motyc closed 10 months ago

motyc commented 2 years ago

Navazuje na #354

od @jnihnat

tam sme sa bavili, ze to neni jednoduche, v aplikaci je pres 700 klicu, co neznamena ze to je 700 miest kde dany kluc je..., Neexistuje ziadny jednoduchy sposob, ako tohle upravit, je potrebne prejst kluc po kluci a upravit to v kode. Zaroven potom, ale bude treba spravit preklady. Priklad: v aplikacii je teraz "Odeslat" - to je i klic a zobrazuje se to i aplikaci, lebo to nema preklad. nahradime to na "common.button.odeslat" to se zacne zobrazovat v prekladech i v aplikaci nekdo pak musi zadat preklad do aplikace. Treba se teda dohodnout, jak to udelat a kdo co udela. zmena v aplikaci je jasna, ale pak zadani noveho prekladu?

jnihnat commented 1 year ago

@motyc k tomu druhemu bodu, chapu to spravne, ze chceme mit co preklad to klic, napr:

: adb/forms.py:40 arch_z/forms.py:122 arch_z/tables.py:28

: arch_z/templates/arch_z/arch_z_detail.html:75 lokalita/tables.py:68

: projekt/forms.py:241 projekt/forms.py:491 projekt/models.py:96

: projekt/templates/projekt/detail.html:286

: projekt/templates/projekt/detail.html:448

msgid "Uživatelské označení"

rozdelit na 10 klicu? Protoze v kodu je 10x

jinak uz mam script ktory najde preklad podle vyskytu v kodu, akurat se jeste musim dohodnout s @pesikj jak to presne udelat, protoze mi pri tom nemuze zasahovat do kodu aby se mi radky neposunuli.

motyc commented 1 year ago

@jnihnat Ano, protože jinak nebude v případě potřeby možné popis pole nijak upřesnit pro dané view. Sice tedy počet klíčů nabyde, ale protože máme název pole vždy i v názvu klíče, je snadné najít všechny najednou a překlad rozkopírovat. Bylo by tedy fajn být v tvorbě názvu klíče konzistentní (vč. buttonů atd.).

jnihnat commented 1 year ago

jo nazev klicu udelam podle umistneni v adresari, napr hore uvedene by bylo: adb.forms.adbForm.label.uzivatelske_oznaceni arch_z.tables.archzTable.label.uzivatelske_oznaceni projekt.models.projekt.label.uzivatelske_oznaceni teda adresar pres bodky, pak trida a pak label/text a nakonec samostatny nazev.

motyc commented 1 year ago

@jnihnat Dokončil jsem revizi textů a stav je následující:

Chyby (identifikováno při průběžném testování)

pesikj commented 1 year ago

@motyc @jnihnat Zde si prosím rozdělte, co by měl kdo opravit. Myslím, že třeba bod divný (výrazně zavádějící) klíč: zvládne opravit @motyc nebo někdo jiný z ústavu.

motyc commented 1 year ago

@pesikj To samozřejmě mohu, jen nevím, do čeho všeho můžu zasahovat, abych něco @jnihnat neměnil pod rukama. Také nevím, zda po změně klíče nemusím udělat ještě něco dalšího, abych nevytvořil nějaký zmatek v překladech.

pesikj commented 1 year ago

Určitě bude potřeba se synchronizovat. Pokud to není priorita, tak navrhuji, aby vám Honza vyčlenil úkoly, na kterých budete pracovat, a Honza se mezitím může věnovat práci na oprávnění. Poté může dokončit zbývající části.

motyc commented 1 year ago

Ono já teď budu také dva týdny víceméně pryč, takže něco na tom dělat mohu případně od září... Spíš tedy třeba může nechat bokem to, co mohu pak dodělat já.

jnihnat commented 1 year ago

Ja to skusim dat pristi tyden dokopy tak aby to bylo udelano v nejakem formatu a aby ste to mohli potom uz upravovat sam ty klice @motyc.

jnihnat commented 1 year ago

@motyc klice nahrane a mnelo by vsetko sedet. Nehovorim ze se nemuze vyskytnout mala chybka ale nic zavratniho. Pokud by ste nahodou objevil cely soubor co je spatne a pod. dejte vedet kouknu na to. Jinak prve dve odrazky som taky uz zapracoval. Na zbytku budu delat a snazit se dokoncit co nejdriv.

motyc commented 1 year ago

@jnihnat Díky moc, vypadá to, že to funguje. Z první dobré jsem narazil jen na to, že chybí řada klíčů pro http://192.168.254.30:8080/upravit-uzivatele/ (některé jsou, některé však ne).

Každopádně podrobně bych to otestoval, až doladíte ty výše referované věci. Jinak překlady budu průběžně editovat, když na něco narazím. Pokud bych někdy měl do administrace nezasahovat, dejte vždy prosím vědět.

Hodně by mi také pomohlo, pokud byste zapnuli to přepínání na EN verzi, abych si snadnou mohl zobrazit klíče.

jnihnat commented 1 year ago

vetsina veci opravena v #1202 co jsem neopravil:

To dodelani prekladu bych teda klidne nechal na vas, kdyz natrafite na nejakou sekci muzete klidne udelat sam a pak pull request a skontroluji to. obecne models.py a tables.py maji verbose_name a forms.py a filters.py maji label.

EN prepinani jsem taky dodelal v #1202

motyc commented 1 year ago

@jnihnat Děkuji moc. Jakmile bude nasazeno, vrhnu se na to a doladím zbytek. Případně budu volat o radu.

motyc commented 1 year ago

Výsledek testu (v0.3.5z14)

jnihnat commented 1 year ago

@motyc opravy v #1316 :

motyc commented 1 year ago

přesunuto do: https://github.com/ARUP-CAS/aiscr-webamcr/issues/558#issuecomment-1806490714

motyc commented 1 year ago

@pesikj @jnihnat Zdá se, že ty úpravy klíčů pro mapy nějak rozbily mapové okno - nejsou v něm vidět žádná tlačítka (v0.3.5z22), související klíče však v Rosettě jsou načteny dobře. Posupoval jsem podle vzoru, ale mohl jsem udělat chybu. Prosím o revizi, co se stalo. Do stejných souborů jinak bude jinak zasahovat i PR od @jiri-bartos-huld (#1322), tak by to bylo dobré zohlednit.

motyc commented 1 year ago

Test v0.3.5z23

jnihnat commented 1 year ago

opravy v #1367 udelal som i preklad na heslare, pokud je EN bere se preklad ze sloupce heslo_en pro notifikace jsem udelal obdobne na text_cs a text_en

motyc commented 1 year ago

@jnihnat To je skvělé! Myslel jsem, že to vyřešíme samostatně. Moc tedy děkuji. Otestuji po nasazení.

motyc commented 1 year ago

Test v0.3.5z47

https://docs.google.com/spreadsheets/d/1it_kr3HCN2vv54SUvn696XaP9tKCgcHnqUCBeg4CGYw/edit#gid=2033966180 - tabulka aktualizována, konkrétní typy chyb vypisuji níže.

EDIT:

jnihnat commented 1 year ago

tedka jsem si vsiml ze u prveho resime dj.forms.createDjForm.typ.label a ne to co je v dropdowne :D Kouknu na to...

if (e.name == "AMČR Piany") { - tohle by malo fungovat opravoval som a overoval u sebe

preklady notifikaci, hodil jsem do DB k modelu nove sloupce text_cs a text_en. Nevsiml jsem si ze to nemame v adminu. Pridam pod Heslare at se to da jednoducho upravovat? Udelam opravu at se vraci ident pokud neni zadny preklad?

L.marker([corX, corY], { icon: pinIconRedPoint }).bindTooltip('Navržený pian').addTo(drawnItems); //'Navržený pian' - taky opraveno a malo by fungovat

Na zvysok sa skusim pozriet este dnes/zajtra

motyc commented 1 year ago

@jnihnat Díky za rychlou reakci. Ještě jsem doplnil čtyři body, co jsem si poznamenal vedle a zapomněl připsat.

jnihnat commented 1 year ago

k e.name - to je dotaz na @jiri-bartos-huld protoze to jsou mapy a tam do toho moc nevidim

Navrzeny pian - v kodu uz je map_translations['TempPIAN'], problem delaly ty [] v kterych ten retezec byl, nekde to treba je a nekde ne...

jnihnat commented 1 year ago

snad finalni opravy v #1372

motyc commented 1 year ago

Test v0.3.5z51

Tak úplně poslední kolo to bohužel nebylo, ale tentokrát už myslím bude. Co zbývá:

Zbylé už je ok.

jnihnat commented 1 year ago

@motyc - překlady heslářů nefungují pro vícestupňové hesláře - nadále platné - potreboval bych vedet kde presne, u PAS zapsat to funguje.

motyc commented 1 year ago

@jnihnat mrkněte do filtračních formulářů, tam to nešlo nikde

jnihnat commented 1 year ago

@jnihnat mrkněte do filtračních formulářů, tam to nešlo nikde

uz jsem na to medzi casem nahodou natrafil... opravim.

motyc commented 11 months ago

Test v0.3.5z61

Tak poslední drobnost:

jnihnat commented 11 months ago

@motyc ted jsem skousel a zobrazuje se spravne jak pro projekt tak pro pas: image

image

motyc commented 11 months ago

@jnihnat Jde o obsah toho hesláře, ne nadpis :)

jnihnat commented 11 months ago

opraveno v #1409 - upravil som nacitani nahledu obrazu v PAS. tedka to neblokuje nacitani javascriptu. Nezobrazili se autocomplete seznamy pokud se nanecetli vsechny obrazky. Jedina nevyhoda je, ze se donacitavaji az pri skrolovani, na druhu stranu nezatazujeme server zbytocnym nacitanim pokud je ani nezobrazime. Dejte vedet ci to nechat a pripadne ci to dodelat na dalsi seznamy, je to jednoducha vec na minutu.

motyc commented 11 months ago

@jnihnat Bezva, na to jsem narazil a určitě bude lepší, když to na sebe čekat nebude. Ono náhledy asi budeme do budoucna nějak cachovat (viz #1090), ale určitě to bude lepší upravit všude.

jnihnat commented 10 months ago

@motyc - tohle je asi na vas na testovani, nebo je tu jeste nejaka chyba co jsem prehledl? Dekuji

pesikj commented 10 months ago

Přehodil jsem to k testování.

motyc commented 10 months ago

@jnihnat @pesikj Bohužel to zůstává stále stejné (totéž i u projektů): image

jnihnat commented 10 months ago

mala chybka, pridam k opravam opravneni na kterych tedka delam