miriris / ukolnicek Goto Github PK

Ahoj Miriam, dobrá práce! Líbí se mi, jak sis zkrátila komponentu ternárním operátorem. Tady to nebylo v zadání, ale pro příště je dobré komponentu oddělit do vlastní složky. Taky jsi dobře definovala funkci renderTasks na řádku 23, akorát je pak zbytečné ji znovu definovat uvnitř callback anonymní funkce EvenListeneru, tam už ji stačí jen zavolat v rámci fetche. Uvnitř téhle podmínky by měl být jenom ten fetch, který přes API získá data a pak zavolá funkci renderTasks .then((data) => renderTasks(data)) nebo zkráceně .then(renderTasks).

Co se týče postavení podmínky, udělala bych to spíš naopak a napsala if (checkUndone.checked). Když není checkUndone.checked už je trochu moc negací. 😄

task1 na začátku byl jenom kvůli testování? :)

Ještě dva tipy:

• řádek 9: Kratší a tradiční způsob jako destrukturovat props je přímo jako parametry funkce export const Task = ({name, due, done}) => {
• řádek: 35: Tady by se spíš hodil Event ('change'), jde nám o změnu stavu checkboxu

Poprosím tě o opravu té anonymní funkce uvnitř EventListeneru a jestli mi napíšeš na Slacku, až to bude pushnuté tady v repozitáři. Díky a měj se hezky!