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

pravopis a další postřehy (formulář, podmínky) #29

Open
klaravytiskova opened this issue Mar 16, 2020 · 6 comments · Fixed by #31
Open

pravopis a další postřehy (formulář, podmínky) #29

klaravytiskova opened this issue Mar 16, 2020 · 6 comments · Fixed by #31
Assignees
Labels
bug Something isn't working ready for dev This user story/bug has been validated by product owner and is ready to be developed.

Comments

@klaravytiskova
Copy link

klaravytiskova commented Mar 16, 2020

formulář:

  1. SOLVED „Vyhledejte vaši adresu“ (í -> i) - vaši má být s krátkým i
  2. po vyplnění a smazání formuláře je většina linek červeně, kromě té u e-mailu (záměr? email není povinný?)
  3. linka u tel. čísla rozjetá
  4. **Transfered to Solve bugs in address sugestion #54 ** adresa - když zadám „Modřany", nabízí 2x "Praha, Modřany“; po zadání ulice se někdy nabídka možností objeví, někdy ne (zatím v tom nevidím systém)

podmínky ochrany osobních údajů:

  1. SOLVED "Osoby mladší 18 let nemohou naší platformu využívat.“ (í -> i) - naší má být s krátkým i
  2. SOLVED "Tyto dobročinné organizace, jsou ve vztahu k osobním údajům, které jim předáváme, povinni tyto osobní údaje použít pouze pro účely kontaktování Vás a koordinaci pomoci a poté je bezodkladně smazat.“ - je navíc čárka vě větě mezi organizace a jsou
  3. SOLVED Vašim, vašich - nekonzistence malá v/velká V
  4. prázdná stránka navíc
@Vratislav Vratislav added bug Something isn't working ready for dev This user story/bug has been validated by product owner and is ready to be developed. labels Mar 16, 2020
@kirscfil kirscfil self-assigned this Mar 16, 2020
@kirscfil
Copy link
Contributor

#31 fixed texts and updated terms and conditions PDF

@kirscfil kirscfil linked a pull request Mar 16, 2020 that will close this issue
@kirscfil kirscfil removed their assignment Mar 16, 2020
@Vratislav
Copy link
Contributor

@all-contributors please add @klaravytiskova for bug

@allcontributors
Copy link
Contributor

@Vratislav

I've put up a pull request to add @klaravytiskova! 🎉

@Vratislav Vratislav self-assigned this Mar 16, 2020
martinhanzik pushed a commit that referenced this issue Mar 18, 2020
@martinhanzik martinhanzik reopened this Mar 18, 2020
@martinhanzik
Copy link
Contributor

Nejsem si jistý, že je všechno fixed, takže reopen než se na to někdo zase podívá.

@klaravytiskova
Copy link
Author

„Vyhledejte vaši adresu“ (í -> i) - vaši má být s krátkým i - na webu stále "vaší"
zbytek vypadá opraven

při kontrole ještě v podmínkách ochrany osobních údajů nalezeno:

  1. "Zpracováváme osobní údaje, které nám poskytnete, jméno, příjmení, telefonní číslo, e-mail a adresa v okolí, které nabízíte pomoc dobročinným organizacím." věta s čárkou za "okolí" nedává smysl, měla by být před -> "...a adresa, v okolí které nabízíte...";
    oddělení "které nám poskytnete" od výčtu čárkou je zvláštní, lépe asi dvojtečka -> "které nám poskytnete: jméno, příjmení..."
  2. pevné mezery u jednopísmenných předložek - aby nezůstávaly na konci řádku (v, s, z, k)
  3. lépe vypadá zarovnané do bloku
  4. "Právním základem pro zpracování osobních údajů je plnění smlouvy. Vy nám svoje osobní
    údaje poskytujete za účelem předání dobročinným organizacím působícím v okolí adresy
    (nebo adres), kterou jste nám poskytli abyste následně mohli vykonávat dobrovolnickou
    činnost." - chybí čárka před "abyste"

@martinhanzik
Copy link
Contributor

Zatím to je jen na https://staging.pomuzeme.si pravděpodobně

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working ready for dev This user story/bug has been validated by product owner and is ready to be developed.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants