Indigo v10 - React 16, Storybook 5

keboola / indigo-ui

Indigo UI library, its CSS and React components. Also Styleguide app showing usages of this library.

https://indigo.keboola.com

MIT License

5 stars 0 forks source link

Indigo v10 - React 16, Storybook 5 #395

Closed natocTo closed 4 years ago

natocTo commented 5 years ago

Pár týdnů zpět jsem se do toho pustil více. Nakonec z toho byl jen ten update kde se vyhodil React 14 #390 Zastavilo se to na tom, že nejnovější Storybook nefunguje s Reactecm 15, i když by mělo.

Teď jsem potřeboval React 16 pro test KBC-UI na React 16 tak jsme to využil. Ale je možné že to takto naráz nebude OK.

natocTo commented 4 years ago

Toto zkusím rebasnout a ještě použít. Pokud bych začal jinde toto by se muselo celé zahodit to by byla škoda. Ten update Storybooku nebyl jen tak. Myslím že ani review pak nemusí být úplně "stopro" aby to zabralo kupu času když to bude jakoby příprava až na tu verzi 10.

ujovlado commented 4 years ago

Ak by to bol problem, kludne to cele zahod a initni storybook nanovo, len s podporou react 16, potrebujeme vlastne odtialto len komponenty. Zda sa mi, ze som to raz tak robil.

natocTo commented 4 years ago

Jo, zatím vypadá že to půjde použít.

ujovlado commented 4 years ago

Bude problem, ked este pribudnu do mastra nejake dalsie veci? Nebudu ale menit Storybook, packages.json alebo tak. Iba pridaju nejaku komponentu alebo nieco zmazu (ikony napr.).

natocTo commented 4 years ago

Neměl by být problém. Klidně se do toho pusť. Já myslel právě že přeskočím sem když v KBC-UI se dělá to review routeru.

natocTo commented 4 years ago

Ono toto teoreticky by mohlo být teda verze 10 který by šla společně s React 16 v KBC-UI ještě do produkce všem vlastně. A verze 11 by mohla být až nakonec ta kde bude čistka. Nebo myslíš blbost?

natocTo commented 4 years ago

Tak snad to bude OK. Bych to právě zkusil jakože:

update react-router na kbc-ui
zkouknout toto a případně vydat verzi 10
nalinkovat verzi 10 do PR v kbc-ui na update React 16 a otestovat
doladit a pustit do produkce
tu v Indigo začít dělat na verzi 11 - stejné jako v10 jen se promažou ty CSS, ikonky, a vše co patří k novému UI

ujovlado commented 4 years ago

Poslem tam este par detailov pred tymto. A toto vydame ako v 10, pripadner upravy sa poriesia ako minor/patch verzie. A v11 bude tak ako si spominal.

ujovlado commented 4 years ago

Idem tomu dat rebase a releasol by som to ako v10.

natocTo commented 4 years ago

Pecka. OK.