inbo / alien-species-portal

Portal for alien and invasive species indicators
MIT License
0 stars 0 forks source link

TRANSLATE #30

Closed SanderDevisscher closed 8 months ago

SanderDevisscher commented 2 years ago

Link to translations to be completed: https://github.com/inbo/alien-species-portal/blob/mockup/alienSpecies/inst/extdata/translations.csv

Originally posted by @mvarewyck in https://github.com/inbo/alien-species-portal/issues/26#issuecomment-1265118348

Je kan dit bestand lokaal bewerken als volgt:

  1. Open github desktop
  2. Selecteer de mockup- branch
  3. fetch/pull branch
  4. klik op show in explorer of klik ctrl+ shift+ f
  5. open het bestand dmv excel
  6. vertaal tekst
  7. sla je wijzigingen op (ctrl+ s)
  8. commit & push je wijzigingen
bramdhondt commented 1 year ago

@SanderDevisscher - Done. Let op: scheidingsteken is onbedoeld gewijzigd van komma naar puntkomma.

mvarewyck commented 1 year ago

Some new translations might be needed: 0d53486b24137ba22597814c7fae2f9d7450a4af

bramdhondt commented 1 year ago

If something is expected, please provide a clear request (of action and by whom).

mvarewyck commented 1 year ago

@SanderDevisscher - Done. Let op: scheidingsteken is onbedoeld gewijzigd van komma naar puntkomma.

Which encoding was used to create the file? I can't read it with UTF-8, UTF-16 or Latin (all special characters are not recognized)

bramdhondt commented 1 year ago

As all fields needed to align well in order to perform the translations, I opened and wrote the csv using Microsoft Excel. Though I had the troubles with the special characters, too. (Correcting them apparently didn't change much...)

mvarewyck commented 1 year ago

@bramdhondt Kan je nog eens door de translation file gaan een aanvullen/wijzigen waar nodig?

bramdhondt commented 1 year ago

@mvarewyck -- Ik ben er grondig doorgegaan. Zie de csv (gepusht).

Aangezien dit enkel werkbaar is in Excel en vanwege het risico op compatibiliteitsproblemen, zeker gezien het hoge aantal leestekens, heb ik de oorspronkelijke versie ook als backup opgeslaan (_translations_BACKUP_22-11-30-14-28_DELETEME.csv) en ook de xlsx behouden (_translationsv22-11-30.xlsx).

For the record: de Franse vertaling van de pathways komen één op één uit een rapport van Dutrieux & Magnier (2017).

bramdhondt commented 1 year ago

@mvarewyck -- Ik ben ook nog eens door de Nederlandstalige pathways gegaan, om die op één lijn te zetten met de Franse en Engelse versies. translations.csv is aldus geüpdatet.

Daarmee denk ik met deze issue klaar te zijn.

mvarewyck commented 1 year ago

@bramdhondt Enkel de formatting van de xlsx file is correct. Ik kan daarmee verder. In de toekomst is het dus best om de xlsx aan te passen en dan (door ons) een csv te laten aanmaken met UTF-8 coding.

mvarewyck commented 1 year ago

@bramdhondt Er zijn nog vertalingen bijgekomen oa. door #28 . Ik heb ze aangevuld in de csv file.

bramdhondt commented 1 year ago

@mvarewyck -- Vertalingen aangebracht in translations_v22-12-20.xlsx . Van daaruit kunt u een nieuwe csv maken. De vorige versie van de xlsx kunt u allicht verwijderen.

mvarewyck commented 1 year ago

@bramdhondt Kan je een nieuwe ronde vertalingen doen? Dit is de laatste versie https://github.com/inbo/alien-species-portal/blob/mockup/alienSpecies/inst/extdata/translations.csv

bramdhondt commented 1 year ago

@mvarewyck De 'laatste versie' waar je naar verwijst bevat nog niet de wijzigingen die ik in het bericht van 20 december 2022 heb doorgevoerd (translations_v22-12-20.xlsx, link). Zie bv. vertalingen voor de continenten, Africa t.e.m. Western Europe; soortstatus, captive t.e.m. widespreadInvasive; ...

Kunnen deze eerst worden ingewerkt, zodat duidelijker wordt waar er nog vertalingen ontbreken?

mvarewyck commented 1 year ago

@bramdhondt Vertalingen zijn up-to-date nu. In de huidige file is er een extra kolom per taal voor 'description'. Hierin kan je extra tekst per figuur plaatsen (in de toekomst)

bramdhondt commented 1 year ago

@mvarewyck - Dag Machteld, ik heb een nieuwe versie van de xlsx gepusht (translations_v23-05-24.xlsx). Ik denk dat ik alles heb. Controleer je even? Tussen nu en midden juli valt mijn beschikbaarheid sterk terug.

mvarewyck commented 1 year ago

@bramdhondt I've added some new labels which need translation. Just search for TODO in this file

bramdhondt commented 1 year ago

@mvarewyck - Dag Machteld, ik heb je vraag bekeken. Zie _translations_v23-09-20_pars.xlsx_ , die ik gepusht heb. De omweg via Excel verhindert conversiefouten aan mijn zijde, en vergemakkelijkt ook het overzicht. Excuses dat dit voor jou wel extra stappen betekent.

In de gele cellen heb ik een vertaling verzorgd. In de blauwe cellen heb ik bestaande tekst verbeterd.

Sommige lijnen lijken wel HTML-code te bevatten, met een foute uitlijning tot gevolg. Hieronder voorbeeld, uit jouw csv. Met die lijnen weet ik niet heel goed wat aan te vatten.

afbeelding

mvarewyck commented 1 year ago

Sommige lijnen lijken wel HTML-code te bevatten, met een foute uitlijning tot gevolg. Hieronder voorbeeld, uit jouw csv. Met die lijnen weet ik niet heel goed wat aan te vatten.

afbeelding

Dit zorgt ervoor dat bepaalde stukken op een nieuwe lijn </br> of in bold <em> worden weergegeven. De zaken tussen accolade, worden berekend in de code en vervangen. Zo zal bvb. {{maxDate}} vervangen worden door de maximum datum berekend obv de data. Het is dus best dat je deze stukjes identiek overneemt in de vertaling, anders is de formatting daar verschillend.

mvarewyck commented 12 months ago

@mvarewyck - Dag Machteld, ik heb je vraag bekeken. Zie _translations_v23-09-20_pars.xlsx_ , die ik gepusht heb. De omweg via Excel verhindert conversiefouten aan mijn zijde, en vergemakkelijkt ook het overzicht. Excuses dat dit voor jou wel extra stappen betekent.

@bramdhondt No problem, I've transferred the translations that were clear from translations_v23-09-20_pars.xlsx. Any update on the special ones (HTML formatted)? We can also do it together during a quick call if that helps.

bramdhondt commented 10 months ago

@mvarewyck - Is de vraag nog geldig? (Zoja, dan zul je me wel even opnieuw moeten aanwijzen waar ik de tekst vindt.)

mvarewyck commented 10 months ago

@mvarewyck - Is de vraag nog geldig? (Zoja, dan zul je me wel even opnieuw moeten aanwijzen waar ik de tekst vindt.)

@bramdhondt Yes, still relevant.

~The translations file is now on the S3 bucket. I don't know how familiar you are with R and whether you have access to the S3 bucket, but instructions on how to update the file with the S3 bucket are here~ We can also do it together during a quick call for the HTML formatted ones.

~I've attached the latest translations file if the above is not working for you. translations.csv~

@SanderDevisscher who will update the translations file in the future? Is it someone with access to the S3 bucket or should we keep the translations file into the git repository?

SanderDevisscher commented 10 months ago

@mvarewyck didn't we agree on placing the translation file here https://github.com/inbo/aspbo/blob/21-upload-files-to-uat/data/output/UAT_direct/translations.csv ? Thus allowing everyone from inbo to do translations. The fis-bot will do the uploading.

mvarewyck commented 10 months ago

@SanderDevisscher Sorry, I missed that one. That's indeed a good solution.

bramdhondt commented 10 months ago

@mvarewyck / @SanderDevisscher - Kun je me nog even verzekeren binnen welke branch ik de laatste versie binnentrek / mijn aanpassingen push ?

SanderDevisscher commented 10 months ago

@bramdhondt momenteel bestaat het translate bestandje enkel in de 21-upload-files-to-uat - branch. Je mag hier je wijzigingen aan toevoegen. Tenzij het over aanzienlijke aanpassingen gaat, dan zou ik wachten tot na de merge van https://github.com/inbo/aspbo/pull/25 en daarna een nieuwe branch openen en PR'en met de uat - branch.

bramdhondt commented 10 months ago

@SanderDevisscher @fleurpet - Update van translations.csv gepusht naar branch 21 (aspbo-repo). Alle velden met "TODO" zijn opgelost.