You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
map: why map, overlay, source, are defined by useRef? זה פלסטר?!
move useStyles functions outside from the react components, utils files or above it
Task.js line 185, task != undefiend...
Task.js line 227-234, it is not readable, please export as much logic as u can from the jsx section.
Standarts
do no use ;
useSelector instead of mapStateToProps
useDispatch instead of mapDispatchToProps
use prettier to arrange spaces (the code is too messy)
there are lots of functions write this way: const handleChange = (e) => { const value = e.target.value; setChange(value) } one line better solution: const handleChange = ({ target: { value } }) => setChange(value)
Critical
useGetTasks & useGetGroupsshould be one hook (suggest name as useFetch), which get url & options.Standarts
const handleChange = (e) => { const value = e.target.value; setChange(value) }one line better solution:const handleChange = ({ target: { value } }) => setChange(value)I was impressed by