Closed pomali closed 2 years ago
Je dobre, ze ten hint sa meni s inputom? Funguje takto nejaky iny komponent? Plus rozmyslam, ci nedat tu validacnu logiku radsej do railsov ako sem do client-side JS. cc @tajovic @filiplikavcan @jsuchal
Nebral som to ako error lebo som predpokladal, že existujú vstupy, ktoré nebudú validované, ale bude možné ich spracovať (úradník im rozumie).
Ako napríklad cudzinci alebo ľudia ktorí sú obeťou nejakej chyby systému.
On Mon, Aug 5, 2019, 10:16 Jano Suchal notifications@github.com wrote:
@jsuchal requested changes on this pull request.
@eronisko https://github.com/eronisko validacia urcite musi ist AJ serverside. Co sa tyka hintu tak podla mna to neni dobre, ze sa to meni. Je to divny pattern co dizajn manual nepozna. Specialne teda errory sa robia inak.
In src/all.js https://github.com/slovensko-digital/navody-frontend/pull/102#discussion_r310484596 :
@@ -11,6 +11,7 @@ import Tabs from './components/tabs/tabs' import SdnHeader from './components/_custom/header/header' import SdnTimeline from './components/_custom/timeline/timeline' import SdnAppearLink from './utilities/appear-link/appear-link' +import SdnInputRodneCislo from './components/_custom/input-rodne-cislo/intput-rodne-cislo'
Ideme po slovensky? Nie som uplne proti len sa pytam.
— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/slovensko-digital/navody-frontend/pull/102?email_source=notifications&email_token=AAJDTR2MHTW2IXPJDYUF5WDQC7OUHA5CNFSM4H6LGJUKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCAQUMUQ#pullrequestreview-270616146, or mute the thread https://github.com/notifications/unsubscribe-auth/AAJDTR5PKGTBUCFF4PE3VU3QC7OUHANCNFSM4H6LGJUA .
V podstate to je normalny text input, avsak pridava hint, po ktorom si pouzivatel moze skontrolovat rodne cislo.
Zvolil som hint a nie error pretoze:
Resolves https://github.com/slovensko-digital/navody-frontend/issues/101
A takto nejako to vyzera (label sa da menit):