Skip to content

improve structure

Jan Koprajda requested to merge mj-improve-structure into master

Created by: marceljager

Jestli jsem dal někoho špatně na review, tak se omlouvam. Naklikal jsem tam lidi, kteří dělali na starších pull requestech a je to hodně změn, tak jsem nechtěl nikoho zapomenout.

Zatim jsem vubec neřešil styly, ani u nových komponent. Takže tam někde mám třeba style atribut, kterýho bych se po nastylování určitě zbavil.

  • přidal jsem eslint, ať ten kód píšeme alespoň trochu stejně + se to teď samo formátuje na save (když si to nastavíte v IDE), takže se to i pohodlnějí píše. Pravidla v .eslintu nejsou pevně daný, je to návrh toho jak se mi to osvědčilo z ostatních projektů. Určitě to neni dokonalý, ale lepší než nic
  • přidal jsem Formik pro práci s formama
  • pro SVG ikony jsem přidal SVGR loader + je načítám jako inline. Díky tomu jde měnit barva ikon přes css.
  • odebral jsem redux. Určitě neni potřeba mít state uplně každé komponenty v reduxu. Projekt s tim byl šíleně složitý. Redux by měl sloužit k uchování dat, které se používají na více stránkách (uživatelská data - email, jmeno..., nastavení aplikace apod). První odkaz co jsem našel na google https://stackoverflow.com/questions/35328056/react-redux-should-all-component-states-be-kept-in-redux-store.

Nakonec jsem změnil strukturu Průvodce. Přijde mi, že nedává smysl, aby každá stránka průvodce měla svojí url. Uživatel musí jít vždycky od začátku, takže různé url nedávají smysl. Zároveň jsem to přesunul na separátní url a neni to hned na homepage -> to se bude hodit, pokud bude v projektu i public web.

Merge request reports

Loading