Duration component migration

janmichek commented 5 years ago

janmichek commented 5 years ago

neviem ci je vhodne davat startTime ako required - naozaj to nemalo v tej povodnej implementacii vyznam?

Dobre ten required vyhodim. Ale zaroven si myslim ze by bylo dobry udelat nejaky empty stav. Alespon plaintextove N/A jako ma Finished komponenta

janmichek commented 5 years ago

Hm, tady mi ten test unika o seknudu oproti lokalnimu spusteni u ty dynamicky komponenty. Zatim jeste resim

ujovlado commented 5 years ago

Vidim, ze pribudla duration prop. No nikde nevidim zmienku preco. Je na to pouzitie?

janmichek commented 5 years ago

Resil jsem to v implementacnim PR

ujovlado commented 5 years ago

Pridal som tam par zmien:

DynamicDuration bolo potrebne zachovat separatne. Inac by sa aj pre staticke datumy pustal setInterval, co je dost fail
Opravil som tam renderovanie. Ak su vsetky props optional, tak pri poslani duration+start time+end time, to renderovalo divne dvojite casy (chapem, ze to nemusi nastat, ale ta komponenta to ma poriesit tak aby na to developer nemusel mysliet)
timeInWords funkcia bola nejako zvlastne skopirovana z kbc-ui - chybala polovica testov + bol vyhodeny argument round - neviem preco.

ujovlado commented 5 years ago

Mergnem to. Ak sa rozhodneme, potom mozeme dat do exportov aj tie helper funkcie timeInWords, durationFrom

janmichek commented 5 years ago

ok, diky Ten 'round' se v kbc nikde nepouzival, naopak se to zaokrouhlovalo pred komponentou, tak jsem to vyhodil.

ujovlado commented 5 years ago

keboola / indigo-ui